[spi] fix issue report RT-Thread#6008

mysterywolf · mysterywolf · commit e270d65f7383 · 2022-06-21T02:08:59.000-04:00
rt_spi_sendrecv8()存在可能未初始化值返回的情况
diff --git a/components/drivers/include/drivers/spi.h b/components/drivers/include/drivers/spi.h
@@ -271,7 +271,7 @@ rt_inline rt_size_t rt_spi_send(struct rt_spi_device *device,
 rt_inline rt_uint8_t rt_spi_sendrecv8(struct rt_spi_device *device,
                                       rt_uint8_t            data)
 {
-    rt_uint8_t value;
+    rt_uint8_t value = 0;
 
     rt_spi_send_then_recv(device, &data, 1, &value, 1);
 
@@ -281,7 +281,7 @@ rt_inline rt_uint8_t rt_spi_sendrecv8(struct rt_spi_device *device,
 rt_inline rt_uint16_t rt_spi_sendrecv16(struct rt_spi_device *device,
                                         rt_uint16_t           data)
 {
-    rt_uint16_t value;
+    rt_uint16_t value = 0;
 
     rt_spi_send_then_recv(device, &data, 2, &value, 2);
 
diff --git a/components/drivers/spi/device_driver_list.txt b/components/drivers/spi/device_driver_list.txt
diff --git a/components/drivers/spi/spi_core.c b/components/drivers/spi/spi_core.c
@@ -300,7 +300,6 @@ rt_size_t rt_spi_transfer(struct rt_spi_device *device,
     else
     {
         rt_set_errno(-RT_EIO);
-
         return 0;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -271,7 +271,7 @@ rt_inline rt_size_t rt_spi_send(struct rt_spi_device *device,`
`271`	`271`	`rt_inline rt_uint8_t rt_spi_sendrecv8(struct rt_spi_device *device,`
`272`	`272`	`rt_uint8_t data)`
`273`	`273`	`{`
`274`		`- rt_uint8_t value;`
	`274`	`+ rt_uint8_t value = 0;`
`275`	`275`
`276`	`276`	`rt_spi_send_then_recv(device, &data, 1, &value, 1);`
`277`	`277`
`@@ -281,7 +281,7 @@ rt_inline rt_uint8_t rt_spi_sendrecv8(struct rt_spi_device *device,`
`281`	`281`	`rt_inline rt_uint16_t rt_spi_sendrecv16(struct rt_spi_device *device,`
`282`	`282`	`rt_uint16_t data)`
`283`	`283`	`{`
`284`		`- rt_uint16_t value;`
	`284`	`+ rt_uint16_t value = 0;`
`285`	`285`
`286`	`286`	`rt_spi_send_then_recv(device, &data, 2, &value, 2);`
`287`	`287`
Original file line number	Diff line number	Diff line change
`@@ -300,7 +300,6 @@ rt_size_t rt_spi_transfer(struct rt_spi_device *device,`
`300`	`300`	`else`
`301`	`301`	`{`
`302`	`302`	`rt_set_errno(-RT_EIO);`
`303`		`-`
`304`	`303`	`return 0;`
`305`	`304`	`}`
`306`	`305`