sysutils/mbgtools: Add error checking for copyin() and copyout() calls

This will be required soon, and the previous lack of such error handling
was a bug.

PR:		275986
Approved by:	antoine
This commit is contained in:
Mark Johnston 2024-01-02 13:07:59 -05:00
parent 5dc186ecf8
commit 0524b7ffff
2 changed files with 18 additions and 0 deletions

View File

@ -1,5 +1,6 @@
PORTNAME= mbgtools
DISTVERSION= fbsd-1.0.0
PORTREVISION= 1
CATEGORIES= sysutils
MASTER_SITES= http://www.meinberg.de/download/drivers/

View File

@ -0,0 +1,17 @@
--- mbglib/common/macioctl.h.orig 2024-01-02 18:05:57 UTC
+++ mbglib/common/macioctl.h
@@ -259,10 +259,12 @@ typedef struct
memcpy( _piob, _pin, _size )
#define _frc_iob_to_pout( _piob, _pout, _size ) \
- copyout( _piob, _pout, _size )
+ if (copyout( _piob, _pout, _size ) != 0) \
+ goto err_inval
#define _frc_iob_from_pin( _piob, _pin, _size ) \
- copyin( _pin, _piob, _size )
+ if (copyin( _pin, _piob, _size ) != 0) \
+ goto err_inval
#define _io_wait_pcps_sec_change( _pddev, _cmd, _type, _pout ) \
goto err_inval