> -----Original Message-----
> From: Burakov, Anatoly
> Sent: Friday, October 5, 2018 10:58 AM

 <snip>

> 
> > +   if (internal_config.process_type == RTE_PROC_PRIMARY) {
> > +           /* if we were secondary process we would try requesting
> > +            * container fd from the primary, but we're the primary
> > +            * process so just exit here
> > +            */
> > +           return -1;
> > +   }
> > +
> > +   p->req = SOCKET_REQ_DEFAULT_CONTAINER;
> > +   strcpy(mp_req.name, EAL_VFIO_MP);
> 
> strlcpy?

Probably, although I admit I only copied this code from SOCKET_REQ_CONTAINER 
handling in this file. Eventually it should be fixed in all places with a 
separate patch.

Thanks,
D.

> 
> Otherwise,
> Reviewed-by: Anatoly Burakov <[email protected]>
> 
> --
> Thanks,
> Anatoly

Reply via email to