Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1700371 > unrolled thread
| Started by | John Johansen <john.johansen@canonical.com> |
|---|---|
| First post | 2017-07-31 22:20 +0200 |
| Last post | 2017-07-31 22:20 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: security/apparmor/label.c:1498]: (style) Redundant condition John Johansen <john.johansen@canonical.com> - 2017-07-31 22:20 +0200
| From | John Johansen <john.johansen@canonical.com> |
|---|---|
| Date | 2017-07-31 22:20 +0200 |
| Subject | Re: security/apparmor/label.c:1498]: (style) Redundant condition |
| Message-ID | <u9pku-2jJ-17@gated-at.bofh.it> |
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
Back to top | Article view | linux.kernel
csiph-web