Re: [PATCH] libmpathutils/uxsock.c: Include string.h for memcpy

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

 



On Mon, 2025-02-17 at 16:03 -0500, Benjamin Marzinski wrote:
> On Mon, Feb 17, 2025 at 12:05:03PM -0800, Khem Raj wrote:
> > Fixes
> > uxsock.c:72:2: error: call to undeclared library function 'memcpy'
> > with type 'void *(void *, const void *, unsigned long)'; ISO C99
> > and later do not support implicit function declarations [-
> > Wimplicit-function-declaration]
> 
> Reviewed-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>

Thanks, it turns out that we have this include already in the queue
branch, but I applied your patch on stable-0.11.y and stable-0.10.y.

Regards
Martin

> > 
> > Signed-off-by: Khem Raj <raj.khem@xxxxxxxxx>
> > Cc: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
> > Cc: Martin Wilck <mwilck@xxxxxxxx>
> > ---
> >  libmpathutil/uxsock.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/libmpathutil/uxsock.c b/libmpathutil/uxsock.c
> > index 2135476d..a474874e 100644
> > --- a/libmpathutil/uxsock.c
> > +++ b/libmpathutil/uxsock.c
> > @@ -8,6 +8,7 @@
> >  #include <stdlib.h>
> >  #include <unistd.h>
> >  #include <stdarg.h>
> > +#include <string.h>
> >  #include <fcntl.h>
> >  #include <sys/ioctl.h>
> >  #include <sys/types.h>
> 






[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux