* [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries
@ 2017-07-06 14:54 Jeff Layton
2017-07-06 15:04 ` Serge E. Hallyn
0 siblings, 1 reply; 6+ messages in thread
From: Jeff Layton @ 2017-07-06 14:54 UTC (permalink / raw)
To: Mimi Zohar, Serge E. Hallyn; +Cc: linux-kernel, linux-security-module
From: Jeff Layton <jlayton@redhat.com>
The init_once routine memsets the whole object to 0, and then
explicitly sets some of the fields to 0 again. Just remove the explicit
initializations.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
---
security/integrity/iint.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/security/integrity/iint.c b/security/integrity/iint.c
index 6fc888ca468e..187b7cb378be 100644
--- a/security/integrity/iint.c
+++ b/security/integrity/iint.c
@@ -153,14 +153,11 @@ static void init_once(void *foo)
struct integrity_iint_cache *iint = foo;
memset(iint, 0, sizeof(*iint));
- iint->version = 0;
- iint->flags = 0UL;
iint->ima_file_status = INTEGRITY_UNKNOWN;
iint->ima_mmap_status = INTEGRITY_UNKNOWN;
iint->ima_bprm_status = INTEGRITY_UNKNOWN;
iint->ima_read_status = INTEGRITY_UNKNOWN;
iint->evm_status = INTEGRITY_UNKNOWN;
- iint->measured_pcrs = 0;
}
static int __init integrity_iintcache_init(void)
--
2.13.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries
2017-07-06 14:54 [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries Jeff Layton
@ 2017-07-06 15:04 ` Serge E. Hallyn
2017-07-06 19:43 ` Mimi Zohar
0 siblings, 1 reply; 6+ messages in thread
From: Serge E. Hallyn @ 2017-07-06 15:04 UTC (permalink / raw)
To: Jeff Layton
Cc: Mimi Zohar, Serge E. Hallyn, linux-kernel, linux-security-module
Quoting Jeff Layton (jlayton@kernel.org):
> From: Jeff Layton <jlayton@redhat.com>
>
> The init_once routine memsets the whole object to 0, and then
> explicitly sets some of the fields to 0 again. Just remove the explicit
> initializations.
>
> Signed-off-by: Jeff Layton <jlayton@redhat.com>
Reviewed-by: Serge Hallyn <serge@hallyn.com>
> ---
> security/integrity/iint.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/security/integrity/iint.c b/security/integrity/iint.c
> index 6fc888ca468e..187b7cb378be 100644
> --- a/security/integrity/iint.c
> +++ b/security/integrity/iint.c
> @@ -153,14 +153,11 @@ static void init_once(void *foo)
> struct integrity_iint_cache *iint = foo;
>
> memset(iint, 0, sizeof(*iint));
> - iint->version = 0;
> - iint->flags = 0UL;
> iint->ima_file_status = INTEGRITY_UNKNOWN;
> iint->ima_mmap_status = INTEGRITY_UNKNOWN;
> iint->ima_bprm_status = INTEGRITY_UNKNOWN;
> iint->ima_read_status = INTEGRITY_UNKNOWN;
> iint->evm_status = INTEGRITY_UNKNOWN;
> - iint->measured_pcrs = 0;
> }
>
> static int __init integrity_iintcache_init(void)
> --
> 2.13.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries
2017-07-06 15:04 ` Serge E. Hallyn
@ 2017-07-06 19:43 ` Mimi Zohar
2017-12-07 12:01 ` Jeff Layton
0 siblings, 1 reply; 6+ messages in thread
From: Mimi Zohar @ 2017-07-06 19:43 UTC (permalink / raw)
To: Serge E. Hallyn, Jeff Layton; +Cc: linux-kernel, linux-security-module
On Thu, 2017-07-06 at 10:04 -0500, Serge E. Hallyn wrote:
> Quoting Jeff Layton (jlayton@kernel.org):
> > From: Jeff Layton <jlayton@redhat.com>
> >
> > The init_once routine memsets the whole object to 0, and then
> > explicitly sets some of the fields to 0 again. Just remove the explicit
> > initializations.
> >
> > Signed-off-by: Jeff Layton <jlayton@redhat.com>
>
> Reviewed-by: Serge Hallyn <serge@hallyn.com>
Thanks, queued.
Mimi
>
> > ---
> > security/integrity/iint.c | 3 ---
> > 1 file changed, 3 deletions(-)
> >
> > diff --git a/security/integrity/iint.c b/security/integrity/iint.c
> > index 6fc888ca468e..187b7cb378be 100644
> > --- a/security/integrity/iint.c
> > +++ b/security/integrity/iint.c
> > @@ -153,14 +153,11 @@ static void init_once(void *foo)
> > struct integrity_iint_cache *iint = foo;
> >
> > memset(iint, 0, sizeof(*iint));
> > - iint->version = 0;
> > - iint->flags = 0UL;
> > iint->ima_file_status = INTEGRITY_UNKNOWN;
> > iint->ima_mmap_status = INTEGRITY_UNKNOWN;
> > iint->ima_bprm_status = INTEGRITY_UNKNOWN;
> > iint->ima_read_status = INTEGRITY_UNKNOWN;
> > iint->evm_status = INTEGRITY_UNKNOWN;
> > - iint->measured_pcrs = 0;
> > }
> >
> > static int __init integrity_iintcache_init(void)
> > --
> > 2.13.0
> --
> To unsubscribe from this list: send the line "unsubscribe linux-security-module" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries
2017-07-06 19:43 ` Mimi Zohar
@ 2017-12-07 12:01 ` Jeff Layton
2017-12-07 14:35 ` Mimi Zohar
0 siblings, 1 reply; 6+ messages in thread
From: Jeff Layton @ 2017-12-07 12:01 UTC (permalink / raw)
To: Mimi Zohar, Serge E. Hallyn; +Cc: linux-kernel, linux-security-module
On Thu, 2017-07-06 at 15:43 -0400, Mimi Zohar wrote:
> On Thu, 2017-07-06 at 10:04 -0500, Serge E. Hallyn wrote:
> > Quoting Jeff Layton (jlayton@kernel.org):
> > > From: Jeff Layton <jlayton@redhat.com>
> > >
> > > The init_once routine memsets the whole object to 0, and then
> > > explicitly sets some of the fields to 0 again. Just remove the explicit
> > > initializations.
> > >
> > > Signed-off-by: Jeff Layton <jlayton@redhat.com>
> >
> > Reviewed-by: Serge Hallyn <serge@hallyn.com>
>
> Thanks, queued.
>
> Mimi
>
Hi Mimi,
I notice that this patch hasn't made the last couple of releases. Was it
dropped for some reason?
Thanks,
Jeff
> >
> > > ---
> > > security/integrity/iint.c | 3 ---
> > > 1 file changed, 3 deletions(-)
> > >
> > > diff --git a/security/integrity/iint.c b/security/integrity/iint.c
> > > index 6fc888ca468e..187b7cb378be 100644
> > > --- a/security/integrity/iint.c
> > > +++ b/security/integrity/iint.c
> > > @@ -153,14 +153,11 @@ static void init_once(void *foo)
> > > struct integrity_iint_cache *iint = foo;
> > >
> > > memset(iint, 0, sizeof(*iint));
> > > - iint->version = 0;
> > > - iint->flags = 0UL;
> > > iint->ima_file_status = INTEGRITY_UNKNOWN;
> > > iint->ima_mmap_status = INTEGRITY_UNKNOWN;
> > > iint->ima_bprm_status = INTEGRITY_UNKNOWN;
> > > iint->ima_read_status = INTEGRITY_UNKNOWN;
> > > iint->evm_status = INTEGRITY_UNKNOWN;
> > > - iint->measured_pcrs = 0;
> > > }
> > >
> > > static int __init integrity_iintcache_init(void)
> > > --
> > > 2.13.0
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-security-module" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
>
--
Jeff Layton <jlayton@kernel.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries
2017-12-07 12:01 ` Jeff Layton
@ 2017-12-07 14:35 ` Mimi Zohar
2017-12-07 15:16 ` Jeff Layton
0 siblings, 1 reply; 6+ messages in thread
From: Mimi Zohar @ 2017-12-07 14:35 UTC (permalink / raw)
To: Jeff Layton, Serge E. Hallyn; +Cc: linux-kernel, linux-security-module
On Thu, 2017-12-07 at 07:01 -0500, Jeff Layton wrote:
> On Thu, 2017-07-06 at 15:43 -0400, Mimi Zohar wrote:
> > On Thu, 2017-07-06 at 10:04 -0500, Serge E. Hallyn wrote:
> > > Quoting Jeff Layton (jlayton@kernel.org):
> > > > From: Jeff Layton <jlayton@redhat.com>
> > > >
> > > > The init_once routine memsets the whole object to 0, and then
> > > > explicitly sets some of the fields to 0 again. Just remove the explicit
> > > > initializations.
> > > >
> > > > Signed-off-by: Jeff Layton <jlayton@redhat.com>
> > >
> > > Reviewed-by: Serge Hallyn <serge@hallyn.com>
> >
> > Thanks, queued.
> >
> > Mimi
> >
>
> Hi Mimi,
>
> I notice that this patch hasn't made the last couple of releases. Was it
> dropped for some reason?
Thanks for the reminder. I accidentally dropped it (and Sascha
Hauer's patch). The subject line is too long. Assuming you don't
object, I'll replace the "get rid of" with "removed" and queue the
patch in the next-queued-testing branch.
Mimi
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries
2017-12-07 14:35 ` Mimi Zohar
@ 2017-12-07 15:16 ` Jeff Layton
0 siblings, 0 replies; 6+ messages in thread
From: Jeff Layton @ 2017-12-07 15:16 UTC (permalink / raw)
To: Mimi Zohar, Serge E. Hallyn; +Cc: linux-kernel, linux-security-module
On Thu, 2017-12-07 at 09:35 -0500, Mimi Zohar wrote:
> On Thu, 2017-12-07 at 07:01 -0500, Jeff Layton wrote:
> > On Thu, 2017-07-06 at 15:43 -0400, Mimi Zohar wrote:
> > > On Thu, 2017-07-06 at 10:04 -0500, Serge E. Hallyn wrote:
> > > > Quoting Jeff Layton (jlayton@kernel.org):
> > > > > From: Jeff Layton <jlayton@redhat.com>
> > > > >
> > > > > The init_once routine memsets the whole object to 0, and then
> > > > > explicitly sets some of the fields to 0 again. Just remove the explicit
> > > > > initializations.
> > > > >
> > > > > Signed-off-by: Jeff Layton <jlayton@redhat.com>
> > > >
> > > > Reviewed-by: Serge Hallyn <serge@hallyn.com>
> > >
> > > Thanks, queued.
> > >
> > > Mimi
> > >
> >
> > Hi Mimi,
> >
> > I notice that this patch hasn't made the last couple of releases. Was it
> > dropped for some reason?
>
> Thanks for the reminder. I accidentally dropped it (and Sascha
> Hauer's patch). The subject line is too long. Assuming you don't
> object, I'll replace the "get rid of" with "removed" and queue the
> patch in the next-queued-testing branch.
>
> Mimi
>
Sounds good.
Thanks,
--
Jeff Layton <jlayton@kernel.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-12-07 15:16 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-06 14:54 [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries Jeff Layton
2017-07-06 15:04 ` Serge E. Hallyn
2017-07-06 19:43 ` Mimi Zohar
2017-12-07 12:01 ` Jeff Layton
2017-12-07 14:35 ` Mimi Zohar
2017-12-07 15:16 ` Jeff Layton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).