[Libguestfs] [PATCH] guestfish: Enable grouping in string lists

[Matthew Booth]

On 12/09/09 11:06, Richard W.M. Jones wrote:
> On Fri, Sep 11, 2009 at 05:54:44PM +0100, Matthew Booth wrote:
>> +      tok = realloc (tok, tok_len + 1);
>> +      if (NULL == tok) { perror ("realloc"); exit (1); }
>
> As before, this leaves tok allocated on the error path.

As discussed on IRC, although this strictly isn't necessary here because 
of the exit(), I'll add this anyway for robustness in case the exit is 
changed later.

>
> [...]
>
> Inclusion of documentation is good, but this patch still needs
> comprehensive tests.

New patch coming up includes comprehensive tests.

Matt
-- 
Matthew Booth, RHCA, RHCSS
Red Hat Engineering, Virtualisation Team

M:       +44 (0)7977 267231
GPG ID:  D33C3490
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490