]> git.ipfire.org Git - thirdparty/grub.git/commitdiff
grub_iso9660_read: Explicitly check read_node return value.
authorVladimir Serbinenko <phcoder@gmail.com>
Sun, 25 Jan 2015 12:19:58 +0000 (13:19 +0100)
committerVladimir Serbinenko <phcoder@gmail.com>
Sun, 25 Jan 2015 19:32:31 +0000 (20:32 +0100)
Not really needed as grub_errno is already checked but is nicer.

Found by: Coverity scan.

grub-core/fs/iso9660.c

index 6a6677337d70e164faa32b5ce63dbab50cb32377..67a67cf40dae4c00355bc4e93d32d449c45ff4c2 100644 (file)
@@ -959,14 +959,15 @@ grub_iso9660_read (grub_file_t file, char *buf, grub_size_t len)
 {
   struct grub_iso9660_data *data =
     (struct grub_iso9660_data *) file->data;
+  grub_err_t err;
 
   /* XXX: The file is stored in as a single extent.  */
   data->disk->read_hook = file->read_hook;
   data->disk->read_hook_data = file->read_hook_data;
-  read_node (data->node, file->offset, len, buf);
+  err = read_node (data->node, file->offset, len, buf);
   data->disk->read_hook = NULL;
 
-  if (grub_errno)
+  if (err || grub_errno)
     return -1;
 
   return len;