[riot-notifications] [RIOT-OS/RIOT] cpu/esp_common: esp_now doesn't call memcpy if iol_len is 0 (#11187)

Martine Lenders notifications at github.com
Fri Mar 15 11:14:45 CET 2019


miri64 commented on this pull request.



> @@ -652,11 +652,11 @@ static int _send(netdev_t *netdev, const iolist_t *iolist)
                   data_len + iolist->iol_len, ESP_NOW_MAX_SIZE_RAW);

But this is only true for GNRC. If another (hypothetical) stack does not necessarily provide this, this can lead to problems. Better document this in the [`esp-now` documentation](https://doc.riot-os.org/esp__now__netdev_8h.html) somewhere.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11187#discussion_r265920130
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190315/3ef5c4d9/attachment.html>


More information about the notifications mailing list