system/freebsd-src
system/freebsd-src
1
0
Fork 0
mirror of https://github.com/freebsd/freebsd-src synced 2024-09-06 09:10:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix a null-pointer dereference and a tautological check in cam_get_device

Browse source 
Reported by:	Coverity
CID:		1017964
MFC after:	3 weeks
Sponsored by:	Spectra Logic Corp
Differential Revision:	https://reviews.freebsd.org/D13184
This commit is contained in:
Alan Somers 2017-12-06 23:24:11 +00:00
parent 762a7f4f5f
commit 0812ee3af2
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=326646
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
lib/libcam/camlib.c
View file

@ -30,6 +30,7 @@ __FBSDID("$FreeBSD$");
#include <sys/types.h>
#include <sys/param.h>
#include <assert.h>
#include <stdio.h>
#include <stdlib.h>
#include <stdint.h>
@ -132,6 +133,9 @@ cam_get_device(const char *path, char *dev_name, int devnamelen, int *unit)
* it so we don't hose the user's string.
*/
newpath = (char *)strdup(path);
if (newpath == NULL)
return (-1);
tmpstr = newpath;
/*
@ -140,8 +144,9 @@ cam_get_device(const char *path, char *dev_name, int devnamelen, int *unit)
if (*tmpstr == '/') {
tmpstr2 = tmpstr;
tmpstr = strrchr(tmpstr2, '/');
if ((tmpstr != NULL) && (*tmpstr != '\0'))
tmpstr++;
/* We know that tmpstr2 contains a '/', so strrchr can't fail */
assert(tmpstr != NULL && *tmpstr != '\0');
tmpstr++;
}
if (*tmpstr == '\0') {