Skip to content

Commit 47f6aec

Browse files
committed
Check parameters in irecv_devices_get_device_by_* API and return error if NULL
1 parent 1d9f1e3 commit 47f6aec

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

src/libirecovery.c

+9
Original file line numberDiff line numberDiff line change
@@ -3225,6 +3225,9 @@ IRECV_API irecv_error_t irecv_devices_get_device_by_client(irecv_client_t client
32253225
#else
32263226
int i = 0;
32273227

3228+
if (!client || !device)
3229+
return IRECV_E_INVALID_INPUT;
3230+
32283231
*device = NULL;
32293232

32303233
if (client->device_info.cpid == 0) {
@@ -3245,6 +3248,9 @@ IRECV_API irecv_error_t irecv_devices_get_device_by_client(irecv_client_t client
32453248
IRECV_API irecv_error_t irecv_devices_get_device_by_product_type(const char* product_type, irecv_device_t* device) {
32463249
int i = 0;
32473250

3251+
if (!product_type || !device)
3252+
return IRECV_E_INVALID_INPUT;
3253+
32483254
*device = NULL;
32493255

32503256
for (i = 0; irecv_devices[i].product_type != NULL; i++) {
@@ -3260,6 +3266,9 @@ IRECV_API irecv_error_t irecv_devices_get_device_by_product_type(const char* pro
32603266
IRECV_API irecv_error_t irecv_devices_get_device_by_hardware_model(const char* hardware_model, irecv_device_t* device) {
32613267
int i = 0;
32623268

3269+
if (!hardware_model || !device)
3270+
return IRECV_E_INVALID_INPUT;
3271+
32633272
*device = NULL;
32643273

32653274
/* lowercase hardware_model string for proper lookup */

0 commit comments

Comments
 (0)