Re: [LTP] [PATCH 06/13]Add missing operators

Garrett Cooper
2011-01-06T19:34:17Z

On Jan 6, 2011, at 3:03 AM, tangchen wrote:

> Signed-off-by: Tang Chen <tangchen@cn.fujitsu.com>
> ---
> testcases/kernel/syscalls/msync/msync04.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/testcases/kernel/syscalls/msync/msync04.c
> b/testcases/kernel/syscalls/msync/msync04.c
> index 3e4b9a4..4e5dccb 100644
> --- a/testcases/kernel/syscalls/msync/msync04.c
> +++ b/testcases/kernel/syscalls/msync/msync04.c
> @@ -93,7 +93,7 @@ int main(int ac, char **av)
> char *msg; /* message returned from parse_opts */
>
> /* Parse standard options given to run the test. */
> - if ((msg = parse_opts(ac, av, NULL, NULL)) NULL)
> + if ((msg = parse_opts(ac, av, NULL, NULL)) != NULL)
> tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg);

Patch is good despite the fact that the description is misleading.
I checked in a different fix though.
Thanks,
-Garrett