All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* 9ead082168053f89: Restore NULL checks before calling kfree().
@ 2005-09-13  0:14 Rusty Russell
  2005-09-13  8:11 ` Christian Limpach
  0 siblings, 1 reply; 2+ messages in thread
From: Rusty Russell @ 2005-09-13  0:14 UTC (permalink / raw
  To: Xen Mailing List; +Cc: Christian Limpach

Hmm, kfree() is defined to work (like free) as a noop on NULL.

Please, when checking in changes, tell us *why*.  Consistency with rest
of code?  Stylistic concerns?  Found a bug?

Thanks!
Rusty.
-- 
A bad analogy is like a leaky screwdriver -- Richard Braakman

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: 9ead082168053f89: Restore NULL checks before calling kfree().
  2005-09-13  0:14 9ead082168053f89: Restore NULL checks before calling kfree() Rusty Russell
@ 2005-09-13  8:11 ` Christian Limpach
  0 siblings, 0 replies; 2+ messages in thread
From: Christian Limpach @ 2005-09-13  8:11 UTC (permalink / raw
  To: Rusty Russell; +Cc: Xen Mailing List

On Tue, Sep 13, 2005 at 10:14:04AM +1000, Rusty Russell wrote:
> Hmm, kfree() is defined to work (like free) as a noop on NULL.
> 
> Please, when checking in changes, tell us *why*.  Consistency with rest
> of code?  Stylistic concerns?  Found a bug?

Consistency with rest of code and stylistic concerns.

    christian

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-09-13  8:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-09-13  0:14 9ead082168053f89: Restore NULL checks before calling kfree() Rusty Russell
2005-09-13  8:11 ` Christian Limpach

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.