Ben Smithurst wrote in list.freebsd-hackers:
> Mark J. Taylor wrote:
> > + dev = malloc(strlen(vnp->dev)+6);
> > + (void)sprintf(dev, "/dev/%s", vnp->dev);
>
> You should be checking that malloc() doesn't return NULL, before trying
> to write into the allocated sp
Ben Smithurst wrote in list.freebsd-hackers:
> Mark J. Taylor wrote:
> > + dev = malloc(strlen(vnp->dev)+6);
> > + (void)sprintf(dev, "/dev/%s", vnp->dev);
>
> You should be checking that malloc() doesn't return NULL, before trying
> to write into the allocated s
Mark J. Taylor wrote:
> + dev = malloc(strlen(vnp->dev)+6);
> + (void)sprintf(dev, "/dev/%s", vnp->dev);
You should be checking that malloc() doesn't return NULL, before trying
to write into the allocated space.
--
Ben Smithurst| PGP: 0x99392F7D
b...@scie
Mark J. Taylor wrote:
> + dev = malloc(strlen(vnp->dev)+6);
> + (void)sprintf(dev, "/dev/%s", vnp->dev);
You should be checking that malloc() doesn't return NULL, before trying
to write into the allocated space.
--
Ben Smithurst| PGP: 0x99392F7D
[EMAIL P
Looks good!
-Matt
Matthew Dillon
:(Since this will be one of my first commits, I'd like to pass it by
:as many people as possible.)
:
:
:This patch makes "vnconfig" like "
Looks good!
-Matt
Matthew Dillon
<[EMAIL PROTECTED]>
:(Since this will be one of my first commits, I'd like to pass it by
:as many people as possible.)
:
:
:This patch mak
(Since this will be one of my first commits, I'd like to pass it by
as many people as possible.)
This patch makes "vnconfig" like "ccdconfig" where you don't have
to specify "/dev/xxx": it will add "/dev/" for you. Same goes
for the "vntab" file.
Also includes "-Wall" cleanup.
[I already know
(Since this will be one of my first commits, I'd like to pass it by
as many people as possible.)
This patch makes "vnconfig" like "ccdconfig" where you don't have
to specify "/dev/xxx": it will add "/dev/" for you. Same goes
for the "vntab" file.
Also includes "-Wall" cleanup.
[I already kno
8 matches
Mail list logo
