Re: [PATCH] Test for sb_getblk return value

On 10/17/05, Glauber de Oliveira Costa <[email protected]> wrote:
> Hi all,
>
> As we discussed earlier, I'm sending a patch that adds test for the
> return value of sb_getblk. This time I focused on the code of the ext2/3
> filesystems. I'm assuming that getblk fails happens due to I/O errors
> and thus returning returning an EIO back wherever it's needed.
>

> -		bh = sb_getblk(inode->i_sb, parent);
> +		if (!(bh = sb_getblk(inode->i_sb, parent))){
> +			err = -EIO;
> +			break;
> +		}

Would be more readable as

		bh = sb_getblk(inode->i_sb, parent);
		if (!bh) {
			err = -EIO;
			break;
		}


--
Jesper Juhl <[email protected]>
Don't top-post  http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please      http://www.expita.com/nomime.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [PATCH] Test for sb_getblk return value
    • From: Glauber de Oliveira Costa <[email protected]>

• References:
  • [PATCH] Test for sb_getblk return value
    • From: [email protected] (Glauber de Oliveira Costa)

• Prev by Date: [Fwd: [PATCH] NFS/RPC/GSS - oops in gss_pipe_release()]
• Next by Date: Re: NFS client problem with kernel 2.6 and SGI IRIX 6.5
• Previous by thread: [PATCH] Test for sb_getblk return value
• Next by thread: Re: [PATCH] Test for sb_getblk return value
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]