On 08/04/2011 08:12 AM, Laine Stump wrote:
> This was detected by Coverity and reported in:
>
>
https://bugzilla.redhat.com/show_bug.cgi?id=728184
>
> Commit d32a46 moved the bit of code that adds the<bridge> element to
> bridge interfaces to the top of the function so that it is always
> done, even if there are no physical devices attached to the bridge
> (because<bridge> is a required element of the grammar). What wasn't
> noticed is that the ERR_NOMEM macro could goto error, and in this case
> nphys and phys_names would be uninitialized.
>
> Fortunately this would only happen on a failure to allocate memory.
> ---
> src/dutil_linux.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
ACK.
Pushed. Thanks!