* Re: security/apparmor/label.c:1498]: (style) Redundant condition
[not found] <AM4PR08MB1156D3B9C875A7F269BAC8E09CA00@AM4PR08MB1156.eurprd08.prod.outlook.com>
@ 2017-07-31 20:11 ` John Johansen
0 siblings, 0 replies; 2+ messages in thread
From: John Johansen @ 2017-07-31 20:11 UTC (permalink / raw)
To: David Binderman, james.l.morris, serge, linux-security-module,
linux-kernel
On 07/17/2017 12:34 AM, David Binderman wrote:
> Hello there,
>
>
> security/apparmor/label.c:1498]: (style) Redundant condition: prev_ns. '!prev_ns || (prev_ns && *prev_ns!=profile->ns)' is equivalent to '!prev_ns || *prev_ns!=profile->ns'
>
> Source code is
>
> if (view != profile->ns &&
> (!prev_ns || (prev_ns && *prev_ns != profile->ns))) {
>
yep, I will add the fix to the next pull request
^ permalink raw reply [flat|nested] 2+ messages in thread
* security/apparmor/label.c:1498]: (style) Redundant condition
@ 2017-07-31 20:11 ` John Johansen
0 siblings, 0 replies; 2+ messages in thread
From: John Johansen @ 2017-07-31 20:11 UTC (permalink / raw)
To: linux-security-module
On 07/17/2017 12:34 AM, David Binderman wrote:
> Hello there,
>
>
> security/apparmor/label.c:1498]: (style) Redundant condition: prev_ns. '!prev_ns || (prev_ns && *prev_ns!=profile->ns)' is equivalent to '!prev_ns || *prev_ns!=profile->ns'
>
> Source code is
>
> if (view != profile->ns &&
> (!prev_ns || (prev_ns && *prev_ns != profile->ns))) {
>
yep, I will add the fix to the next pull request
--
To unsubscribe from this list: send the line "unsubscribe linux-security-module" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-31 20:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <AM4PR08MB1156D3B9C875A7F269BAC8E09CA00@AM4PR08MB1156.eurprd08.prod.outlook.com>
2017-07-31 20:11 ` security/apparmor/label.c:1498]: (style) Redundant condition John Johansen
2017-07-31 20:11 ` John Johansen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.