d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add sanity testing against maximum sane lengths for device paths for

Browse Source 
loader and kernel.

CID: 1383608
Sponsored by: Netflix
This commit is contained in:
imp 2017-12-12 19:26:24 +00:00
parent 45bef17ee1
commit 8749eeff28
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
usr.sbin/efibootmgr/efibootmgr.c
View File

@ -650,8 +650,14 @@ make_boot_var(const char *label, const char *loader, const char *kernel, const c
kerneldp = NULL;
}
llen = efidp_size(loaderdp);
if (llen > MAX_DP_LEN)
errx(1, "Loader path too long.");
klen = efidp_size(kerneldp);
if (klen > MAX_DP_LEN)
errx(1, "Kernel path too long.");
dp = malloc(llen + klen);
if (dp == NULL)
errx(1, "Can't allocate memory for new device paths");
memcpy(dp, loaderdp, llen);
if (kerneldp != NULL)
memcpy((char *)dp + llen, kerneldp, klen);