upstream/opnsense-src
1
0
Fork 0
mirror of https://github.com/opnsense/src.git synced 2026-06-06 15:22:34 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

compat_freebsd4: Check for errors from subyte() in freebsd4_uname()

Browse source 
This is in preparation for adding a __result_use_check annotation to
copyin() and related functions.

Reviewed by:	imp, kib, jhb
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D43099
This commit is contained in:
Mark Johnston 2023-12-25 20:39:21 -05:00
parent a01ff11cb7
commit 4f35450ce5
1 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
sys/kern/kern_xxx.c
View file

@ -324,7 +324,9 @@ freebsd4_uname(struct thread *td, struct freebsd4_uname_args *uap)
1, 0, 0, 0, 0);
if (error)
return (error);
subyte( uap->name->sysname + sizeof(uap->name->sysname) - 1, 0);
error = subyte(uap->name->sysname + sizeof(uap->name->sysname) - 1, 0);
if (error)
return (EFAULT);
name[1] = KERN_HOSTNAME;
len = sizeof uap->name->nodename;
@ -332,7 +334,9 @@ freebsd4_uname(struct thread *td, struct freebsd4_uname_args *uap)
1, 0, 0, 0, 0);
if (error)
return (error);
subyte( uap->name->nodename + sizeof(uap->name->nodename) - 1, 0);
error = subyte(uap->name->nodename + sizeof(uap->name->nodename) - 1, 0);
if (error)
return (EFAULT);
name[1] = KERN_OSRELEASE;
len = sizeof uap->name->release;
@ -340,7 +344,9 @@ freebsd4_uname(struct thread *td, struct freebsd4_uname_args *uap)
1, 0, 0, 0, 0);
if (error)
return (error);
subyte( uap->name->release + sizeof(uap->name->release) - 1, 0);
error = subyte(uap->name->release + sizeof(uap->name->release) - 1, 0);
if (error)
return (EFAULT);
/*
name = KERN_VERSION;
@ -373,7 +379,9 @@ freebsd4_uname(struct thread *td, struct freebsd4_uname_args *uap)
1, 0, 0, 0, 0);
if (error)
return (error);
subyte( uap->name->machine + sizeof(uap->name->machine) - 1, 0);
error = subyte(uap->name->machine + sizeof(uap->name->machine) - 1, 0);
if (error)
return (EFAULT);
return (0);
}