Re: [PATCH 04/12] HPPFS: fix access to ppos and file->f_pos

On Sun, Sep 18, 2005 at 04:09:51PM +0200, Paolo 'Blaisorblade' Giarrusso wrote:
>  static ssize_t read_proc(struct file *file, char *buf, ssize_t count,
>  			 loff_t *ppos, int is_user)
>  {
> @@ -228,17 +237,21 @@ static ssize_t read_proc(struct file *fi
>  	if (read == NULL)
>  		return -EINVAL;
>  
> +	WARN_ON(is_user != (ppos != NULL));

Eww....  Why not pass the right ppos in all cases?
-
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 04/12] HPPFS: fix access to ppos and file->f_pos
    • From: Blaisorblade <[email protected]>

• References:
  • [PATCH 04/12] HPPFS: fix access to ppos and file->f_pos
    • From: "Paolo 'Blaisorblade' Giarrusso" <[email protected]>

• Prev by Date: Re: [uml-devel] Re: [patch 1/2] Fixup symlink function pointers for hppfs [for 2.6.13]
• Next by Date: Re: [PATCH 12/12] HPPFS: fix nameidata handling
• Previous by thread: [PATCH 04/12] HPPFS: fix access to ppos and file->f_pos
• Next by thread: Re: [PATCH 04/12] HPPFS: fix access to ppos and file->f_pos
• Index(es):
  • Date
  • Thread

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