From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek =?ISO-8859-1?Q?Beh=FAn?= <marek.behun@nic.cz>
Date: Fri, 6 Oct 2017 16:28:34 +0200
Subject: [U-Boot] [PATCH 2/4] fs: btrfs: Fix usage of uninitialized
 variables
In-Reply-To: <99f56d19-544c-4604-4f82-bd7a9a1e45b6@comcast.net>
References: <20171006130459.29127-1-marek.behun@nic.cz>
 <20171006130459.29127-2-marek.behun@nic.cz>
 <99f56d19-544c-4604-4f82-bd7a9a1e45b6@comcast.net>
Message-ID: <20171006162834.45c19c4e@dellmb.labs.office.nic.cz>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de

On Fri, 6 Oct 2017 06:54:34 -0700
"J. William Campbell" <jwilliamcampbell@comcast.net> wrote:

> Hi Marek,
>          If the searched key is not found, isn't it an error to use
> the res value AT ALL? Setting it to 0 may cover up the warning, but
> the function shouldn't actually use it for anything. If it does, it
> is modifying an element that doesn't match the search key.

Hi Bill,

the res variable is just an indicator if an error occured.
In these functions if no key is found, it is not an error (for example
if no key is found in btrfs_readdir, it just means the directory is
empty, which is not an error).

Marek

> 
> Best Regards,
> Bill Campbell