Browse Source

Remove some potential security flaw

Godzil 7 years ago
parent
commit
2dae6e611c
5 changed files with 9 additions and 9 deletions
  1. 2 2
      about.c
  2. 1 1
      cgi_func.c
  3. 2 2
      cgibase.c
  4. 2 2
      commander.c
  5. 2 2
      modprofil.c

+ 2 - 2
about.c

@@ -9,7 +9,7 @@
 #include "cgi_theme_gest.h"
 #include "test_serre.h"
 
-char cgi_BaseName[100];
+char cgi_BaseName[256];
 
 int main(int argc, char *argv[])
 {
@@ -18,7 +18,7 @@ int main(int argc, char *argv[])
 
     cgitheme_init();
 
-    strcpy(cgi_BaseName,argv[0]);
+    strncpy(cgi_BaseName,argv[0], 256);
 
 
     if ( (cgi_getparam("restart")) != NULL )

+ 1 - 1
cgi_func.c

@@ -17,7 +17,7 @@
 #define TRUE 1
 //#define DEBUG
 
-extern char cgi_BaseName[100];
+extern char cgi_BaseName[256];
 
 int cgi_started = FALSE;
 int cgi_paramparsed = FALSE;

+ 2 - 2
cgibase.c

@@ -8,7 +8,7 @@
 #include "cgi_theme_gest.h"
 #include "test_serre.h"
 
-char cgi_BaseName[100];
+char cgi_BaseName[256];
 
 int main(int argc, char *argv[])
 {
@@ -17,7 +17,7 @@ int main(int argc, char *argv[])
 
     cgitheme_init();
 
-    strcpy(cgi_BaseName,argv[0]);
+    strncpy(cgi_BaseName,argv[0], 256);
 
 
     if ( (cgi_getparam("restart")) != NULL )

+ 2 - 2
commander.c

@@ -36,7 +36,7 @@ typedef struct sockaddr_in SOCKADDR_IN;
 #include "cgi_theme_gest.h"
 #include "test_serre.h"
 
-char cgi_BaseName[100];
+char cgi_BaseName256;
 
 
 int main(int argc, char *argv[])
@@ -57,7 +57,7 @@ int main(int argc, char *argv[])
 
     cgitheme_init();
 
-    strcpy(cgi_BaseName,argv[0]);
+    strncpy(cgi_BaseName,argv[0], 256);
 
 
     if ( (cgi_getparam("restart")) != NULL )

+ 2 - 2
modprofil.c

@@ -17,7 +17,7 @@
 #include "cgi_theme_gest.h"
 #include "test_serre.h"
 
-char cgi_BaseName[100];
+char cgi_BaseName[256];
 
 int isnumber(char *value)
 {
@@ -49,7 +49,7 @@ int main(int argc, char *argv[])
 
     cgitheme_init();
 
-    strcpy(cgi_BaseName,argv[0]);
+    strncpy(cgi_BaseName,argv[0], 256);
 
 
     if ( (cgi_getparam("restart")) != NULL )