[libvirt] [PATCH] conf: virDomainDefValidateInternal prohibit some characters in shmem name
Pavel Hrdina
phrdina at redhat.com
Tue Jul 10 11:54:50 UTC 2018
On Tue, Jul 10, 2018 at 01:45:33PM +0200, Peter Krempa wrote:
> On Tue, Jul 10, 2018 at 13:37:34 +0200, Simon Kobyda wrote:
> > XML shmem name will not include characters '/', '.' or '..', as shmem name
> > is used in a path
> > https://bugzilla.redhat.com/show_bug.cgi?id=1192400
> > ---
> > src/conf/domain_conf.c | 13 +++++++++++++
> > 1 file changed, 13 insertions(+)
> >
> > diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> > index 7ab2953d83..3f580525bb 100644
> > --- a/src/conf/domain_conf.c
> > +++ b/src/conf/domain_conf.c
> > @@ -6107,6 +6107,8 @@ virDomainDefLifecycleActionValidate(const virDomainDef *def)
> > static int
> > virDomainDefValidateInternal(const virDomainDef *def)
> > {
> > + size_t i;
> > +
> > if (virDomainDefCheckDuplicateDiskInfo(def) < 0)
> > return -1;
> >
> > @@ -6136,6 +6138,17 @@ virDomainDefValidateInternal(const virDomainDef *def)
> > return -1;
> > }
> >
> > + for (i = 0; i < def->nshmems; i++) {
> > + if (strchr(def->shmems[i]->name, '/'))
> > + return -1;
> > +
> > + if (strchr(def->shmems[i]->name, '.'))
> > + return -1;
> > +
> > + if (strstr(def->shmems[i]->name, ".."))
> > + return -1;
>
> This is dead code since a single dot is rejected by the above
> statement.
>
> Also you need to report an error, since the user would not know what is
> wrong here.
Instead of strchr you can use virStringHasChars(), so in this case:
virStringHasChars(def->shmems[i]->name, "/.")
Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180710/44576651/attachment-0001.sig>
More information about the libvir-list
mailing list