ecore-con - fix possible unterminated buffers
authorCarsten Haitzler (Rasterman) <raster@rasterman.com>
Fri, 13 Dec 2013 07:27:04 +0000 (16:27 +0900)
committerCarsten Haitzler (Rasterman) <raster@rasterman.com>
Fri, 13 Dec 2013 12:26:05 +0000 (21:26 +0900)
this should fix CID 1039725 and CID 1039724

src/lib/ecore_con/ecore_con_local.c

index df86334..54dde17 100644 (file)
@@ -91,7 +91,10 @@ ecore_con_local_connect(Ecore_Con_Server *svr,
         if (svr->port < 0)
           {
              if (svr->name[0] == '/')
-               strncpy(buf, svr->name, sizeof(buf));
+               {
+                  strncpy(buf, svr->name, sizeof(buf) - 1);
+                  buf[sizeof(buf) - 1] = 0;
+               }
              else
                snprintf(buf, sizeof(buf), "/tmp/.ecore_service|%s", svr->name);
           }
@@ -108,8 +111,10 @@ ecore_con_local_connect(Ecore_Con_Server *svr,
           }
      }
    else if ((svr->type & ECORE_CON_TYPE) == ECORE_CON_LOCAL_ABSTRACT)
-     strncpy(buf, svr->name,
-             sizeof(buf));
+     {
+        strncpy(buf, svr->name, sizeof(buf) - 1);
+        buf[sizeof(buf) - 1] = 0;
+     }
 
    svr->fd = socket(AF_UNIX, SOCK_STREAM, 0);
    if (svr->fd < 0)
@@ -251,8 +256,10 @@ ecore_con_local_listen(
                    svr->port);
      }
    else if ((svr->type & ECORE_CON_TYPE) == ECORE_CON_LOCAL_ABSTRACT)
-     strncpy(buf, svr->name,
-             sizeof(buf));
+     {
+        strncpy(buf, svr->name, sizeof(buf) - 1);
+        buf[sizeof(buf) - 1] = 0;
+     }
 
    pmode = umask(mask);
 start: