----- Original Message -----
> From: "Wanlong Gao" <[email protected]>
> To: "LTP" <[email protected]>
> Cc: "Cyril Hrubis" <[email protected]>, [email protected], "Wanlong Gao" 
> <[email protected]>
> Sent: Wednesday, 4 December, 2013 3:35:22 AM
> Subject: [PATCH] sendmsg01: fix test for invalid to buffer
> 
> We should set a valid to buffer length to test for invalid to
> buffer, and invalid buffer should return EFAULT.
> 
> Signed-off-by: Wanlong Gao <[email protected]>

Looks good to me.
Reviewed-by: Jan Stancek <[email protected]>

Regards,
Jan

> ---
>  testcases/kernel/syscalls/sendmsg/sendmsg01.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/testcases/kernel/syscalls/sendmsg/sendmsg01.c
> b/testcases/kernel/syscalls/sendmsg/sendmsg01.c
> index c385631..5c79a45 100644
> --- a/testcases/kernel/syscalls/sendmsg/sendmsg01.c
> +++ b/testcases/kernel/syscalls/sendmsg/sendmsg01.c
> @@ -227,9 +227,9 @@ struct test_case_t tdat[] = {
>        .msg = &msgdat,
>        .flags = 0,
>        .to = (struct sockaddr *)-1,
> -      .tolen = -1,
> +      .tolen = sizeof(struct sockaddr),
>        .retval = -1,
> -      .experrno = EINVAL,
> +      .experrno = EFAULT,
>        .setup = setup1,
>        .cleanup = cleanup1,
>        .desc = "invalid to buffer"},
> --
> 1.8.5
> 
> 

------------------------------------------------------------------------------
Sponsored by Intel(R) XDK 
Develop, test and display web and hybrid apps with a single code base.
Download it for free now!
http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to