Re: [PATCH] libsepol: add missing word separators in error message

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 4, 2024 at 3:05 PM James Carter <jwcart2@xxxxxxxxx> wrote:
>
> On Wed, Dec 4, 2024 at 9:06 AM Christian Göttsche
> <cgoettsche@xxxxxxxxxxxxx> wrote:
> >
> > From: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
> >
> > The concatenation of the multi line error messages misses a separating
> > space between the boundary words.
> >
> > Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
>
> Acked-by: James Carter <jwcart2@xxxxxxxxx>
>
Merged.
Thanks,
Jim

> > ---
> >  libsepol/src/write.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/libsepol/src/write.c b/libsepol/src/write.c
> > index 4ef98449..89c80e7e 100644
> > --- a/libsepol/src/write.c
> > +++ b/libsepol/src/write.c
> > @@ -231,13 +231,13 @@ static int avtab_write_item(policydb_t * p,
> >                 return POLICYDB_ERROR;
> >         if ((p->policyvers < POLICYDB_VERSION_XPERMS_IOCTL) &&
> >                         (cur->key.specified & AVTAB_XPERMS)) {
> > -               ERR(fp->handle, "policy version %u does not support extended"
> > +               ERR(fp->handle, "policy version %u does not support extended "
> >                                 "permissions rules and one was specified", p->policyvers);
> >                 return POLICYDB_ERROR;
> >         }
> >
> >         if (!policydb_has_cond_xperms_feature(p) && (cur->key.specified & AVTAB_XPERMS) && conditional) {
> > -               ERR(fp->handle, "policy version %u does not support extended"
> > +               ERR(fp->handle, "policy version %u does not support extended "
> >                                 "permissions rules in conditional policies and one was specified", p->policyvers);
> >                 return POLICYDB_ERROR;
> >         }
> > --
> > 2.45.2
> >
> >





[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux