Nul terminate

Might be a better idea to implement strlcpy()

diff --git a/modules/access/smb.c b/modules/access/smb.c
index e45a466f2904dcbe3b163caa7fedfacd0729f402..5d889e012fb0449befbe7d788f38523ca4126612 100644 (file)
--- a/modules/access/smb.c
+++ b/modules/access/smb.c
@@ -458,14 +458,18 @@ static void Win32AddConnection( access_t *p_access, char *psz_path,
 
     /* Find out server and share names */
     strncpy( psz_server, psz_path, sizeof( psz_server ) );
+    psz_server[sizeof (psz_server) - 1] = '\0';
     psz_share[0] = 0;
     psz_parser = strchr( psz_path, '/' );
     if( psz_parser )
     {
         char *psz_parser2 = strchr( ++psz_parser, '/' );
         if( psz_parser2 )
+        {
             strncpy( psz_share, psz_parser, sizeof( psz_share ) );
-    }
+            psz_parse[sizeof (psz_parse) - 1] = '\0';
+        }
+   }
 
     sprintf( psz_remote, "\\\\%s\\%s", psz_server, psz_share );
     net_resource.lpRemoteName = psz_remote;