Re: [PATCH] apparmor: remove dead code

2018-08-23 Thread kbuild test robot
Hi Gustavo, Thank you for the patch! Yet something to improve: [auto build test ERROR on security/next] [also build test ERROR on v4.18 next-20180822] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] apparmor: remove dead code

2018-08-23 Thread kbuild test robot
Hi Gustavo, Thank you for the patch! Yet something to improve: [auto build test ERROR on security/next] [also build test ERROR on v4.18 next-20180822] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] apparmor: remove dead code

2018-08-23 Thread Gustavo A. R. Silva
On 8/23/18 1:21 PM, John Johansen wrote: > On 08/23/2018 06:42 AM, Gustavo A. R. Silva wrote: > > thank you for the patch, but a fix for this issue was pushed to apparmor-next > yesterday > That's great. Good to know. Thanks -- Gustavo

Re: [PATCH] apparmor: remove dead code

2018-08-23 Thread Gustavo A. R. Silva
On 8/23/18 1:21 PM, John Johansen wrote: > On 08/23/2018 06:42 AM, Gustavo A. R. Silva wrote: > > thank you for the patch, but a fix for this issue was pushed to apparmor-next > yesterday > That's great. Good to know. Thanks -- Gustavo

Re: [PATCH] apparmor: remove dead code

2018-08-23 Thread John Johansen
On 08/23/2018 06:42 AM, Gustavo A. R. Silva wrote: thank you for the patch, but a fix for this issue was pushed to apparmor-next yesterday > Due to commit fb5841091f28 ("apparmor: remove no-op permission check > in policy_unpack"), there is some leftover code. > > Coverity reports this issue

Re: [PATCH] apparmor: remove dead code

2018-08-23 Thread John Johansen
On 08/23/2018 06:42 AM, Gustavo A. R. Silva wrote: thank you for the patch, but a fix for this issue was pushed to apparmor-next yesterday > Due to commit fb5841091f28 ("apparmor: remove no-op permission check > in policy_unpack"), there is some leftover code. > > Coverity reports this issue

[PATCH] apparmor: remove dead code

2018-08-23 Thread Gustavo A. R. Silva
Due to commit fb5841091f28 ("apparmor: remove no-op permission check in policy_unpack"), there is some leftover code. Coverity reports this issue as Structurally dead code. Fix this by removing such code. Addresses-Coverity-ID: 1472998 ("Structurally dead code") Fixes: fb5841091f28 ("apparmor:

[PATCH] apparmor: remove dead code

2018-08-23 Thread Gustavo A. R. Silva
Due to commit fb5841091f28 ("apparmor: remove no-op permission check in policy_unpack"), there is some leftover code. Coverity reports this issue as Structurally dead code. Fix this by removing such code. Addresses-Coverity-ID: 1472998 ("Structurally dead code") Fixes: fb5841091f28 ("apparmor: