[libvirt] [PATCH] virsh-network: Avoid possible NULL deref in cmdNetworkDHCPLeases
Peter Krempa
pkrempa at redhat.com
Wed Jun 8 12:17:28 UTC 2016
On Wed, Jun 08, 2016 at 13:53:05 +0200, Michal Privoznik wrote:
> Problem is, localtime_r() returns a pointer to converted time or
> NULL in case of an error. But checking the glibc sources, error
> will occur iff a NULL has been passed as an either of arguments
> the function takes. But GCC fails to see that:
>
> ../../tools/virsh-network.c: In function 'cmdNetworkDHCPLeases':
> ../../tools/virsh-network.c:1370:12: error: potential null pointer dereference [-Werror=null-dereference]
> ts = *localtime_r(&expirytime_tmp, &ts);
> ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> cc1: all warnings being treated as errors
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
> tools/virsh-network.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
ACK
More information about the libvir-list
mailing list