dpif-linux: Avoid null dereference if all ports disappear.
authorBen Pfaff <blp@nicira.com>
Mon, 14 Jul 2014 20:17:05 +0000 (13:17 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 15 Jul 2014 17:13:49 +0000 (10:13 -0700)
commitbf1038beea0ee28cf9a2232df91a3b16ed085b52
tree590566145d3815628710a8a0587b6c64e2ae8846
parentd95dc459e7da7e20838c2a88695bd46396b50508
dpif-linux: Avoid null dereference if all ports disappear.

When dpif_linux_refresh_channels() refreshes the set of channels when
the number of handlers changes, it destroys all the dpif's channels and
sets dpif->uc_array_size to 0.  If the port dump later in the function
turns up no ports (which generally indicates a bug), then no channels will
be allocated and thus dpif->uc_array_size will remain 0 and 'channels' will
be null in each handler.  This is self-consistent, at least, but
dpif_linux_port_get_pid__() was still willing in this situation to
try to access element 0 of the set of channels, dereferencing a null
pointer.

This fixes the problem.

I encountered this while looking at a bug that I had introduced during
development that caused the port dump to always be empty.  It would be
difficult to encounter in normal use.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Joe Stringer <joestringer@nicira.com>
lib/dpif-linux.c