Skip to content

Commit e369b3e

Browse files
committed
Fix possibility of a buffer overflow
1 parent b7095cd commit e369b3e

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

version.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@
66
#define MAIN_VERSION_PATCH 0
77

88
#define MAIN_VERSION_BETA 0
9-
#define MAIN_VERSION_DATE "2023-12-29"
9+
#define MAIN_VERSION_DATE "2023-12-31"
1010

1111
#endif

vncserver.c

+4-4
Original file line numberDiff line numberDiff line change
@@ -184,9 +184,9 @@ int main(int argc, char **argv) {
184184

185185
// Preset values from environment variables (However, the values specified in the arguments have priority.)
186186
if (getenv("VNC_SERVERNAME"))
187-
strcpy(VNC_SERVERNAME, getenv("VNC_SERVERNAME"));
187+
snprintf(VNC_SERVERNAME, sizeof(VNC_SERVERNAME), "%s", getenv("VNC_SERVERNAME"));
188188
if (getenv("VNC_PASSWORD"))
189-
strcpy(VNC_PASSWORD, getenv("VNC_PASSWORD"));
189+
snprintf(VNC_PASSWORD, sizeof(VNC_PASSWORD), "%s", getenv("VNC_PASSWORD"));
190190
if (getenv("VNC_PORT"))
191191
VNC_PORT = atoi(getenv("VNC_PORT"));
192192

@@ -206,11 +206,11 @@ int main(int argc, char **argv) {
206206
break;
207207
case 'n':
208208
i++;
209-
strcpy(VNC_SERVERNAME,argv[i]);
209+
snprintf(VNC_SERVERNAME, sizeof(VNC_SERVERNAME), "%s", argv[i]);
210210
break;
211211
case 'p':
212212
i++;
213-
strcpy(VNC_PASSWORD,argv[i]);
213+
snprintf(VNC_PASSWORD, sizeof(VNC_PASSWORD), "%s", argv[i]);
214214
break;
215215
case 'f':
216216
i++;

0 commit comments

Comments
 (0)