Re: [RFC][patch 2/7] v9fs: VFS file and directory operations (2.0-rc6)

From: Pekka Enberg
Date: Tue May 24 2005 - 02:18:00 EST


Hi,

On 5/24/05, ericvh@xxxxxxxxx <ericvh@xxxxxxxxx> wrote:
> +static ssize_t
> +v9fs_file_write(struct file *filp, const char __user * data,
> + size_t count, loff_t * offset)
> +{
> + int ret = -1;
> + char *buffer;
> +
> + buffer = kmalloc(count, GFP_KERNEL);
> + if (buffer == NULL) {
> + BUG();

I think simply returning -ENOMEM is sufficient. BUG seems way too
aggressive. (Found this in other places as well.)

> + return -ENOMEM;
> + }

Pekka
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/