projects
/
bertos.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix undefined behaviour.
[bertos.git]
/
bertos
/
cpu
/
arm
/
drv
/
eth_at91.c
diff --git
a/bertos/cpu/arm/drv/eth_at91.c
b/bertos/cpu/arm/drv/eth_at91.c
index 5d365b20ae712b720c8a7d43816b300fd0cf52de..529d7af4e5473b51045ef1ec435e108abc9edc7a 100644
(file)
--- a/
bertos/cpu/arm/drv/eth_at91.c
+++ b/
bertos/cpu/arm/drv/eth_at91.c
@@
-425,7
+425,11
@@
ssize_t eth_getFrame(uint8_t *buf, size_t len)
rd_len += EMAC_RX_BUFSIZ;
else
rd_len += len - rd_len;
rd_len += EMAC_RX_BUFSIZ;
else
rd_len += len - rd_len;
- ASSERT(rx_buf_tab[rx_buf_idx].addr & RXBUF_OWNERSHIP);
+ if (UNLIKELY(!(rx_buf_tab[rx_buf_idx].addr & RXBUF_OWNERSHIP)))
+ {
+ LOG_INFO("bad frame found\n");
+ return 0;
+ }
rx_buf_tab[rx_buf_idx].addr &= ~RXBUF_OWNERSHIP;
if (++rx_buf_idx >= EMAC_RX_DESCRIPTORS)
rx_buf_idx = 0;
rx_buf_tab[rx_buf_idx].addr &= ~RXBUF_OWNERSHIP;
if (++rx_buf_idx >= EMAC_RX_DESCRIPTORS)
rx_buf_idx = 0;