Re: [tyndur-devel] [PATCH 1/2] libc: Fehlerbehandlung fuer init_execute()

[Kevin Wolf <mail@xxxxxxxxxxxxx>]

Am Freitag, 29. Mai 2009 21:53 schrieb Antoine Kaufmann:
> ! libc: Fehlerbehandlung fuer init_execute(""). Bis jetzt gab das schoen
>         rot.
>
> Signed-off-by: Antoine Kaufmann <toni@xxxxxxxxxx>
> ---
>  src/modules/lib/init.c |    8 +++++---
>  1 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/src/modules/lib/init.c b/src/modules/lib/init.c
> index 59ce869..cf44305 100644
> --- a/src/modules/lib/init.c
> +++ b/src/modules/lib/init.c
> @@ -111,10 +111,12 @@ pid_t init_execute(const char* cmd)
>
>      // Pruefen ob es sich um einen Pfad handelt
>      abs_path = io_get_absolute_path(program);
> -    if (access(abs_path, R_OK) != 0) {
> +    if (!abs_path || (access(abs_path, R_OK) != 0)) {
>          // Den absoluten Pfad freigeben
> -        free(abs_path);
> -        abs_path = NULL;
> +        if (abs_path) {
> +            free(abs_path);
> +            abs_path = NULL;
> +        }

Das if ist unnötig.

Dürfte aber was sinnvolles tun, also rein damit. ;-)