d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Re-order terms to avoid potential pointer overflow, and remove one

Browse Source 
more potential buffer overflow.

Submitted by:	bde
This commit is contained in:
Eivind Eklund 1997-03-18 16:09:27 +00:00
parent 753da60320
commit 43ec585267
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/libtermcap/tgoto.c
View File

@ -174,7 +174,12 @@ casedot:
* to be the successor of tab.
*/
do {
strcat(added, oncol ? (BC ? BC : "\b") : UP);
const char *extra;
extra = oncol ? (BC ? BC : "\b") : UP;
if (strlen(added) + strlen(extra) >= 10)
return ("OVERFLOW");
strcat(added, extra);
which++;
} while (which == '\n');
}
@ -215,7 +220,7 @@ casedot:
goto toohard;
}
}
if (dp+strlen(added)+1 > &result[MAXRETURNSIZE])
if (strlen(added) >= &result[MAXRETURNSIZE] - dp)
return ("OVERFLOW");
strcpy(dp, added);
return (result);