On 11/21/2017 10:16 PM, Wolfgang Denk wrote:
Dear Heinrich,

In message <20171121184503.3193-1-xypron.g...@gmx.de> you wrote:
If 'file' cannot be allocated due to an out of memory
situation, do not dereference it.

When debugging this patch also avoids a misleading message
"cannot find next direntry, error %d" in case of an out of
memory situation. It is sufficent to write
"%s: Error, no memory for malloc!\n" in this case.

Reported-by: Alex Sadovsky <nable.mainin...@googlemail.com>
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
  fs/ubifs/ubifs.c | 5 ++---
  1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/fs/ubifs/ubifs.c b/fs/ubifs/ubifs.c
index 4465523d5f..313dee0579 100644
--- a/fs/ubifs/ubifs.c
+++ b/fs/ubifs/ubifs.c
@@ -403,8 +403,7 @@ static int ubifs_finddir(struct super_block *sb, char 
*dirname,
        dir = kzalloc(sizeof(struct inode), 0);
        if (!file || !dentry || !dir) {
                printf("%s: Error, no memory for malloc!\n", __func__);
-               err = -ENOMEM;
-               goto out;
+               goto out_nomem;
        }
dir->i_sb = sb;
@@ -424,7 +423,6 @@ static int ubifs_finddir(struct super_block *sb, char 
*dirname,
                err = PTR_ERR(dent);
                goto out;
        }
-
        file->f_pos = key_hash_flash(c, &dent->key);
        file->private_data = dent;
@@ -463,6 +461,7 @@ out: out_free:
        kfree(file->private_data);
+out_nomem:
        free(file);
        free(dentry);
        free(dir);

Should you not keep the "err = -ENOMEM;" setting?  Otherwise there
is no indivcation that an error happened.

err is a local variable it is not returned by the function. It is only used to show a debug message which does not make sense in this case.

The "inidcation" is returning 0 and
printf("%s: Error, no memory for malloc!\n", __func__)

Best regards

Heinrich


Best regards,

Wolfgang Denk

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to