[riot-notifications] [RIOT-OS/RIOT] net/sock_util: Accept NULL pointers in urlsplit (#11677)

Leandro Lanzieri notifications at github.com
Tue Jun 11 22:28:21 CEST 2019


leandrolanzieri commented on this pull request.



>  
-    size_t pathlen = strlen(pathstart);
-    if (pathlen) {
-        if (pathlen > SOCK_URLPATH_MAXLEN - 1) {
-            return -EOVERFLOW;
+    if (urlpath) {
+        size_t pathlen = strlen(pathstart);
+        if (pathlen) {

I can't see the issue here, isn't it valid to pass 0 length to memcpy?

-- 
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/11677#discussion_r292645229
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190611/c022c12d/attachment.html>


More information about the notifications mailing list