[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [lvm-devel] [PATCH 05/29] Test *buf for NULL



On Thu, Nov 25, 2010 at 11:55:09AM +0100, Zdenek Kabelac wrote:
> As we reallocate *buf in CLVMD_CMD_TEST we need to test for NULL
> to print status to avoid printing to NULL buffer.
> (realloc() == NULL & status != 0)

Ack for now.

> CHECKME:
> General question - what is supposed to be written in *retlen for  *buf == NULL?
 
The callers initialise it to 0 so it should stay 0 and the callers ought to
cope...

Alasdair


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]