Raspberries/mraa
Private
Public Access
2
0
Fork 0
Code Activity

gpio: only call ferror in free() case as dev has already been free'd by swig code

Browse Source 
Signed-off-by: Brendan Le Foll <brendan.le.foll@intel.com>
This commit is contained in:
Brendan Le Foll
2014-05-06 18:12:53 +01:00
parent d82cae1312
commit ecc2c92027
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/gpio/gpio.c
View File

@@ -211,15 +211,16 @@ maa_gpio_unexport(maa_gpio_context *dev)
fwrite(bu, sizeof(char), length, unexport_f); fwrite(bu, sizeof(char), length, unexport_f);
fclose(unexport_f); fclose(unexport_f);
if (ferror(dev->value_fp) != 0) {
return MAA_ERROR_INVALID_RESOURCE;
}
return MAA_SUCCESS; return MAA_SUCCESS;
} }
maa_result_t maa_result_t
maa_gpio_close(maa_gpio_context *dev) maa_gpio_close(maa_gpio_context *dev)
{ {
if (ferror(dev->value_fp) != 0) {
return MAA_ERROR_INVALID_RESOURCE;
}
maa_gpio_unexport(dev); maa_gpio_unexport(dev);
free(dev); free(dev);
return MAA_SUCCESS; return MAA_SUCCESS;