Re: Proper error handling on NULL pointers

2009-10-30 Thread Jeff Mahoney
On 10/26/2009 05:14 AM, Chris Mason wrote: On Fri, Oct 23, 2009 at 02:08:48PM -0400, Jeff Mahoney wrote: On 10/22/2009 06:15 AM, Andi Drebes wrote: I don't know what is the developer plan to fix that - apparently it's not in the high-priority list (but it must be certainly in the priority

Re: Proper error handling on NULL pointers

2009-10-30 Thread Chris Mason
On Fri, Oct 30, 2009 at 09:27:36AM -0400, Jeff Mahoney wrote: On 10/26/2009 05:14 AM, Chris Mason wrote: On Fri, Oct 23, 2009 at 02:08:48PM -0400, Jeff Mahoney wrote: On 10/22/2009 06:15 AM, Andi Drebes wrote: I don't know what is the developer plan to fix that - apparently it's not in

Re: Proper error handling on NULL pointers

2009-10-23 Thread Jeff Mahoney
On 10/22/2009 06:15 AM, Andi Drebes wrote: I don't know what is the developer plan to fix that - apparently it's not in the high-priority list (but it must be certainly in the priority list, anyone who gets out of memory using btrfs will have some chances of getting an oops [...]). I'd really

Re: Proper error handling on NULL pointers

2009-10-22 Thread Andi Drebes
I don't know what is the developer plan to fix that - apparently it's not in the high-priority list (but it must be certainly in the priority list, anyone who gets out of memory using btrfs will have some chances of getting an oops [...]). I'd really appreciate to see a TODO section somewhere

Re: Proper error handling on NULL pointers

2009-10-22 Thread Diego Calleja
On Jueves 22 Octubre 2009 12:15:59 Andi Drebes escribió: I'd really appreciate to see a TODO section somewhere in the wiki. [..] There's one (needs updating): http://btrfs.wiki.kernel.org/index.php/Development_timeline Also, http://btrfs.wiki.kernel.org/index.php/Project_ideas has some ideas.

Proper error handling on NULL pointers

2009-10-19 Thread Andi Drebes
Hi! I recently posted a message that addresses the usage of BUG_ON for memory shortages concerning btrfs_alloc_path() (see [1]): ... path = btrfs_alloc_path(); BUG_ON(!path); ret = btrfs_search_slot(NULL, root, search_key, path, 0, 0); ... It seems that