[systemd-devel] [PATCH] fix uninitialized access to var in error path

Zbigniew Jędrzejewski-Szmek zbyszek at in.waw.pl
Fri May 16 15:06:15 PDT 2014


On Fri, May 16, 2014 at 08:25:46PM +0200, Ronny Chevalier wrote:
> ---
>  src/libsystemd/sd-bus/bus-objects.c | 2 +-
>  src/libsystemd/sd-bus/sd-bus.c      | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/libsystemd/sd-bus/bus-objects.c b/src/libsystemd/sd-bus/bus-objects.c
> index 51d4a62..dbb04e5 100644
> --- a/src/libsystemd/sd-bus/bus-objects.c
> +++ b/src/libsystemd/sd-bus/bus-objects.c
> @@ -1602,7 +1602,7 @@ static int add_object_vtable_internal(
>                  sd_bus_object_find_t find,
>                  void *userdata) {
>  
> -        sd_bus_slot *s;
> +        sd_bus_slot *s = NULL;
>          struct node_vtable *i, *existing = NULL;
>          const sd_bus_vtable *v;
>          struct node *n;
> diff --git a/src/libsystemd/sd-bus/sd-bus.c b/src/libsystemd/sd-bus/sd-bus.c
> index ec2843f..1f1a4d3 100644
> --- a/src/libsystemd/sd-bus/sd-bus.c
> +++ b/src/libsystemd/sd-bus/sd-bus.c
> @@ -2821,7 +2821,7 @@ _public_ int sd_bus_add_match(
>  
>          struct bus_match_component *components = NULL;
>          unsigned n_components = 0;
> -        sd_bus_slot *s;
> +        sd_bus_slot *s = NULL;
>          int r = 0;
>  
Heh, I wrote an identical patch yesterday, even with the same title, and
forgot to push. When I saw your patch I was confused for a moment :)
I've pushed my version now.

Zbyszek


More information about the systemd-devel mailing list