PmodALS driver changes

Added checking inputs of its functions.
Variable used to read from SPI changed from 1 byte to 4. SPI is in 2 bytes modes and produced stack overwrite.

Author: Manwlis

code/CM7/Drivers/PmodALS/Src/pmodals.c

@@ -6,6 +6,9 @@
 
 pmodals_device_t pmodals_create_handle( const void* const phy_handle , float Vcc , float Rload )
 {
+	if( phy_handle == NULL )
+		return (pmodals_device_t ){0};
+
 	return (pmodals_device_t ){
 		.phy_handle = phy_handle ,
 		.Vcc = Vcc ,
@@ -18,18 +21,21 @@ pmodals_device_t pmodals_create_handle( const void* const phy_handle , float Vcc
 
 HAL_StatusTypeDef pmodals_get_lux( const pmodals_device_t* const device , float* const lux )
 {
-	uint8_t data;
+	if( device == NULL || lux == NULL )
+		return HAL_ERROR;
+
+	uint32_t data = 0;
 #if PMODALS_OS == NO_OS
 	HAL_StatusTypeDef rv = HAL_SPI_Receive( (SPI_HandleTypeDef*) device->phy_handle , (uint8_t *)&data , 1 , HAL_MAX_DELAY );
 #else
-	HAL_StatusTypeDef rv = HAL_SPI_Receive_IT( (SPI_HandleTypeDef*) device->phy_handle , (uint8_t *)&data , 1 );
+	HAL_StatusTypeDef rv = HAL_SPI_Receive_IT( (SPI_HandleTypeDef*) device->phy_handle , (uint8_t*)&data , 1 );
 #endif
 
 	if( rv != HAL_OK )
 		return rv;
 
 #if PMODALS_OS == FREE_RTOS
-	osThreadFlagsWait( SPI_MEM_IT_FLAG , osFlagsWaitAny , osWaitForever );
+	osThreadFlagsWait( SPI_MEM_IT_FLAG | SPI_ERR_IT_FLAG , osFlagsWaitAny , osWaitForever );
 #endif
 
 	*lux = ( (float)data * device->Vcc / device->adc_max_value / device->Rload ) * device->TEMT6000_transfer_function;