Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 30 Jan 2023 17:57:57 -0300
From:      Renato Botelho <garga@FreeBSD.org>
To:        Phil Shafer <phil@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: ef7d10fde4f6 - main - Add missing check for xo_parse_args return code.
Message-ID:  <f78b5f78-4891-1f32-d0d3-c2af0f350bf2@FreeBSD.org>
In-Reply-To: <202301302044.30UKi82I030877@gitrepo.freebsd.org>
References:  <202301302044.30UKi82I030877@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 30/01/23 17:44, Phil Shafer wrote:
> The branch main has been updated by phil:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=ef7d10fde4f650c7e9ef207af53d5f116b689826
> 
> commit ef7d10fde4f650c7e9ef207af53d5f116b689826
> Author:     Phil Shafer <phil@FreeBSD.org>
> AuthorDate: 2023-01-30 20:43:01 +0000
> Commit:     Phil Shafer <phil@FreeBSD.org>
> CommitDate: 2023-01-30 20:43:01 +0000
> 
>      Add missing check for xo_parse_args return code.
> ---
>   usr.bin/iscsictl/iscsictl.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/usr.bin/iscsictl/iscsictl.c b/usr.bin/iscsictl/iscsictl.c
> index 258b4f7c28ac..5b2a62874163 100644
> --- a/usr.bin/iscsictl/iscsictl.c
> +++ b/usr.bin/iscsictl/iscsictl.c
> @@ -745,6 +745,9 @@ main(int argc, char **argv)
>   	struct target *targ;
>   
>   	argc = xo_parse_args(argc, argv);
> +        if (argc < 0)
> +                exit(1);
> +

Code seems to be indented with spaces instead of tabs
-- 
Renato Botelho




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?f78b5f78-4891-1f32-d0d3-c2af0f350bf2>