[PATCH] lib: fwts_pipeio: fwts_pipe_exec() should return FWTS_ERROR instead of -1

Keng-Yu Lin kengyu at canonical.com
Wed Dec 12 03:06:48 UTC 2012


On Fri, Dec 7, 2012 at 9:33 PM, Colin King <colin.king at canonical.com> wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> fwts_pipe_exec() should be returning FWTS_ERROR instead -1. Note that
> FWTS_ERROR is defined as -1, so this makes no real functional difference.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/lib/src/fwts_pipeio.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/lib/src/fwts_pipeio.c b/src/lib/src/fwts_pipeio.c
> index cb6bd1e..68abee9 100644
> --- a/src/lib/src/fwts_pipeio.c
> +++ b/src/lib/src/fwts_pipeio.c
> @@ -143,7 +143,7 @@ int fwts_pipe_exec(const char *command, fwts_list **list)
>         int     ret;
>
>         if ((fd = fwts_pipe_open(command, &pid)) < 0)
> -               return -1;
> +               return FWTS_ERROR;
>
>         text = fwts_pipe_read(fd, &len);
>         *list = fwts_list_from_text(text);
> --
> 1.8.0
>
Acked-by: Keng-Yu Lin <kengyu at canonical.com>



More information about the fwts-devel mailing list