Fix some more null pointer dereferences

This commit is contained in:
Jef Driesen 2017-03-07 21:38:22 +01:00
parent 8735071fac
commit 57ffb2ba7a
2 changed files with 5 additions and 3 deletions

View File

@ -498,7 +498,7 @@ cressi_leonardo_extract_dives (dc_device_t *abstract, const unsigned char data[]
}
if (previous && previous != footer + 2) {
ERROR (abstract->context, "Profiles are not continuous (0x%04x 0x%04x 0x%04x).", header, footer, previous);
ERROR (context, "Profiles are not continuous (0x%04x 0x%04x 0x%04x).", header, footer, previous);
free (buffer);
return DC_STATUS_DATAFORMAT;
}

View File

@ -346,7 +346,7 @@ dc_usbhid_read (dc_usbhid_t *usbhid, void *data, size_t size, size_t *actual)
if (usbhid == NULL) {
status = DC_STATUS_INVALIDARGS;
goto out;
goto out_invalidargs;
}
#if defined(HAVE_LIBUSB) && !defined(__APPLE__)
@ -369,6 +369,7 @@ dc_usbhid_read (dc_usbhid_t *usbhid, void *data, size_t size, size_t *actual)
out:
HEXDUMP (usbhid->context, DC_LOGLEVEL_INFO, "Read", (unsigned char *) data, nbytes);
out_invalidargs:
if (actual)
*actual = nbytes;
@ -387,7 +388,7 @@ dc_usbhid_write (dc_usbhid_t *usbhid, const void *data, size_t size, size_t *act
if (usbhid == NULL) {
status = DC_STATUS_INVALIDARGS;
goto out;
goto out_invalidargs;
}
#if defined(HAVE_LIBUSB) && !defined(__APPLE__)
@ -410,6 +411,7 @@ dc_usbhid_write (dc_usbhid_t *usbhid, const void *data, size_t size, size_t *act
out:
HEXDUMP (usbhid->context, DC_LOGLEVEL_INFO, "Write", (unsigned char *) data, nbytes);
out_invalidargs:
if (actual)
*actual = nbytes;