Skip to content

Commit 563978a

Browse files
committed
socks4 upstream: add safety check for hostname length
1 parent 7ea9f80 commit 563978a

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/reqs.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1263,6 +1263,8 @@ connect_to_upstream_proxy(struct conn_s *connptr, struct request_s *request)
12631263
memcpy(&buff[4], "\0\0\0\1" /* socks4a fake ip */
12641264
"\0" /* user */, 5);
12651265
len = strlen(request->host);
1266+
if(len>255)
1267+
return -1;
12661268
memcpy(&buff[9], request->host, len+1);
12671269
if (9+len+1 != safe_write(connptr->server_fd, buff, 9+len+1))
12681270
return -1;

0 commit comments

Comments
 (0)