mbox series

[v3,0/3] Fix bug caused by default umask

Message ID 1585553269-119544-1-git-send-email-zou_wei@huawei.com
Headers show
Series Fix bug caused by default umask | expand

Message

Samuel Zou March 30, 2020, 7:27 a.m. UTC
v3:
The case is only successful when system default umask is 0022.
When default umask is 0077 or other values, should clear umask by umask(0) in setup().

Zou Wei (3):
  security/dirtyc0w: Fix bug caused by default umask
  syscalls/fstat: Fix bug caused by default umask
  syscalls/statx01: Fix bug caused by default umask

 testcases/kernel/security/dirtyc0w/dirtyc0w.c | 2 ++
 testcases/kernel/syscalls/fstat/fstat02.c     | 2 ++
 testcases/kernel/syscalls/statx/statx01.c     | 2 ++
 3 files changed, 6 insertions(+)

--
2.6.2

Comments

Xiao Yang March 30, 2020, 7:34 a.m. UTC | #1
On 2020/3/30 15:27, Zou Wei wrote:
> v3:
> The case is only successful when system default umask is 0022.
> When default umask is 0077 or other values, should clear umask by umask(0) in setup().
Hi Zou,

Thanks for your patch set.
Did you find any other cases related with the same issue?

Thanks,
Xiao Yang
>
> Zou Wei (3):
>    security/dirtyc0w: Fix bug caused by default umask
>    syscalls/fstat: Fix bug caused by default umask
>    syscalls/statx01: Fix bug caused by default umask
>
>   testcases/kernel/security/dirtyc0w/dirtyc0w.c | 2 ++
>   testcases/kernel/syscalls/fstat/fstat02.c     | 2 ++
>   testcases/kernel/syscalls/statx/statx01.c     | 2 ++
>   3 files changed, 6 insertions(+)
>
> --
> 2.6.2
>
>
Samuel Zou March 30, 2020, 8:22 a.m. UTC | #2
Hi Yang,

I downloaded the development version of ltp two weeks ago, ran the cases in the runtest directory, and analyzed all the failed cases. 
I found that these 3 cases have the umask issue. 

Due to the limited time invested by individuals, it is possible that other cases with the same issue will not be discovered.
I will analyze the other failed cases that have not been concluded further in the next few weeks.

Thanks for your suggestions and reviews.

Best Regards
Zou Wei
-----邮件原件-----
发件人: Xiao Yang [mailto:yangx.jy@cn.fujitsu.com] 
发送时间: 2020年3月30日 15:34
收件人: Zouwei (Samuel) <zou_wei@huawei.com>
抄送: ltp@lists.linux.it
主题: Re: [LTP] [PATCH v3 0/3] Fix bug caused by default umask

On 2020/3/30 15:27, Zou Wei wrote:
> v3:
> The case is only successful when system default umask is 0022.
> When default umask is 0077 or other values, should clear umask by umask(0) in setup().
Hi Zou,

Thanks for your patch set.
Did you find any other cases related with the same issue?

Thanks,
Xiao Yang
>
> Zou Wei (3):
>    security/dirtyc0w: Fix bug caused by default umask
>    syscalls/fstat: Fix bug caused by default umask
>    syscalls/statx01: Fix bug caused by default umask
>
>   testcases/kernel/security/dirtyc0w/dirtyc0w.c | 2 ++
>   testcases/kernel/syscalls/fstat/fstat02.c     | 2 ++
>   testcases/kernel/syscalls/statx/statx01.c     | 2 ++
>   3 files changed, 6 insertions(+)
>
> --
> 2.6.2
>
>
Xiao Yang March 30, 2020, 8:59 a.m. UTC | #3
On 2020/3/30 16:22, Zouwei (Samuel) wrote:
> Hi Yang,
> 
> I downloaded the development version of ltp two weeks ago, ran the cases in the runtest directory, and analyzed all the failed cases.
> I found that these 3 cases have the umask issue.
> 
> Due to the limited time invested by individuals, it is possible that other cases with the same issue will not be discovered.
> I will analyze the other failed cases that have not been concluded further in the next few weeks.
Hi Zou,

Thanks, I have pushed the patch set.
Perhaps, you can fix all same umask issues by a single patch in future.

Thanks,
Xiao Yang
> 
> Thanks for your suggestions and reviews.
> 
> Best Regards
> Zou Wei
> -----邮件原件-----
> 发件人: Xiao Yang [mailto:yangx.jy@cn.fujitsu.com]
> 发送时间: 2020年3月30日 15:34
> 收件人: Zouwei (Samuel)<zou_wei@huawei.com>
> 抄送: ltp@lists.linux.it
> 主题: Re: [LTP] [PATCH v3 0/3] Fix bug caused by default umask
> 
> On 2020/3/30 15:27, Zou Wei wrote:
>> v3:
>> The case is only successful when system default umask is 0022.
>> When default umask is 0077 or other values, should clear umask by umask(0) in setup().
> Hi Zou,
> 
> Thanks for your patch set.
> Did you find any other cases related with the same issue?
> 
> Thanks,
> Xiao Yang
>>
>> Zou Wei (3):
>>     security/dirtyc0w: Fix bug caused by default umask
>>     syscalls/fstat: Fix bug caused by default umask
>>     syscalls/statx01: Fix bug caused by default umask
>>
>>    testcases/kernel/security/dirtyc0w/dirtyc0w.c | 2 ++
>>    testcases/kernel/syscalls/fstat/fstat02.c     | 2 ++
>>    testcases/kernel/syscalls/statx/statx01.c     | 2 ++
>>    3 files changed, 6 insertions(+)
>>
>> --
>> 2.6.2
>>
>>
> 
> 
> 
> 
>