Commit 15b9925

committed

Append null terminator after string copying

The buffer length passed to the callback is the actual length minus 1, so 'buflen' is not out-of-bounds but exactly the last byte of the buffer. Change-Id: I63a2bb1928c80f5c78dac7133133ad2c89f82e5f

1 parent a3539f1 commit 15b9925Copy full SHA for 15b9925

File tree

1 file changed

-0

lines changed

web.c

1 file changed

-0

lines changed

`‎web.c`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,7 @@ int web_eventmux(char *buf, size_t buflen)`
`260`	`260`	`char *buffer = "HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\n\r\n";`
`261`	`261`	`web_send(web_connfd, buffer);`
`262`	`262`	`strncpy(buf, p, buflen);`
	`263`	`+ buf[buflen] = '0円';`
`263`	`264`	`free(p);`
`264`	`265`	`close(web_connfd);`
`265`	`266`	`return strlen(buf);`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 15b9925

File tree

1 file changed

1 file changed

`‎web.c`

0 commit comments