bughz/3proxy
1
0
Fork 0
mirror of https://github.com/3proxy/3proxy.git synced 2025-02-23 10:35:40 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Memory safety issues on configuration reload

Browse Source 
memcpy for potentially overlapped regions replaced with memmove
This commit is contained in:
z3APA3A 2015-11-28 22:56:20 +03:00
parent 4ab2da228b
commit 6c187d167b
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/3proxy.c
View File

@ -313,7 +313,7 @@ int parsestr (unsigned char *str, unsigned char **argm, int nitems, unsigned cha
len = 0;
if(argm[argc]!=(incbegin+1)) {
len = (int)strlen((char *)argm[argc]);
memcpy(buf+*inbuf, argm[argc], len);
memmove(buf+*inbuf, argm[argc], len);
}
if((res = read(fd, buf+*inbuf+len, STRINGBUF-(1+len))) <= 0) {
perror((char *)incbegin+1);
@ -1873,7 +1873,7 @@ int readconfig(FILE * fp){
struct commands * cm;
int res = 0;
if( !(buf = myalloc(bufsize)) || ! (argv = myalloc(NPARAMS * sizeof(unsigned char *) + 1)) ) {
if( !(buf = myalloc(bufsize)) || ! (argv = myalloc((NPARAMS + 1) * sizeof(unsigned char *))) ) {
fprintf(stderr, "No memory for configuration");
return(10);
}