1
0
mirror of https://git.FreeBSD.org/ports.git synced 2024-10-27 21:29:02 +00:00
freebsd-ports/net/mopd/files/patch-process.c
David E. O'Brien 8f7ba5aeb3 String paranoia and security fixes from OpenBSD: prevent a remote buffer
overflow and remote syslog() exploits.

Obtained from:	OpenBSD
2000-08-29 00:49:08 +00:00

82 lines
2.4 KiB
C

--- mopd/process.c 1996/09/21 19:12:26 1.2
+++ mopd/process.c 2000/07/04 23:46:23 1.8
@@ -267,7 +267,7 @@
dllist[slot].a_lseek = 0;
dllist[slot].count = 0;
- if (dllist[slot].dl_bsz >= 1492)
+ if ((dllist[slot].dl_bsz >= 1492) || (dllist[slot].dl_bsz == 0))
dllist[slot].dl_bsz = 1492;
if (dllist[slot].dl_bsz == 1030) /* VS/uVAX 2000 needs this */
dllist[slot].dl_bsz = 1000;
@@ -350,10 +350,10 @@
close(dllist[slot].ldfd);
dllist[slot].ldfd = 0;
dllist[slot].status = DL_STATUS_FREE;
- sprintf(line,
+ snprintf(line,sizeof(line),
"%x:%x:%x:%x:%x:%x Load completed",
dst[0],dst[1],dst[2],dst[3],dst[4],dst[5]);
- syslog(LOG_INFO, line);
+ syslog(LOG_INFO, "%s", line);
return;
}
@@ -438,7 +438,7 @@
{
u_char tmpc;
u_short moplen;
- u_char pfile[17], mopcode;
+ u_char pfile[129], mopcode;
char filename[FILENAME_MAX];
char line[100];
int i,nfd,iindex;
@@ -487,6 +487,8 @@
rpr_pgty = mopGetChar(pkt,index); /* Program Type */
tmpc = mopGetChar(pkt,index); /* Software ID Len */
+ if (tmpc > sizeof(pfile) - 1)
+ return;
for (i = 0; i < tmpc; i++) {
pfile[i] = mopGetChar(pkt,index);
pfile[i+1] = '\0';
@@ -513,31 +515,32 @@
bcopy((char *)src, (char *)(dl_rpr->eaddr), 6);
mopProcessInfo(pkt,index,moplen,dl_rpr,trans);
- sprintf(filename,"%s/%s.SYS", MOP_FILE_PATH, pfile);
+ snprintf(filename,sizeof(filename),
+ "%s/%s.SYS", MOP_FILE_PATH, pfile);
if ((mopCmpEAddr(dst,dl_mcst) == 0)) {
if ((nfd = open(filename, O_RDONLY, 0)) != -1) {
close(nfd);
mopSendASV(src, ii->eaddr, ii, trans);
- sprintf(line,
+ snprintf(line,sizeof(line),
"%x:%x:%x:%x:%x:%x (%d) Do you have %s? (Yes)",
src[0],src[1],src[2],
src[3],src[4],src[5],trans,pfile);
} else {
- sprintf(line,
+ snprintf(line,sizeof(line),
"%x:%x:%x:%x:%x:%x (%d) Do you have %s? (No)",
src[0],src[1],src[2],
src[3],src[4],src[5],trans,pfile);
}
- syslog(LOG_INFO, line);
+ syslog(LOG_INFO, "%s", line);
} else {
if ((mopCmpEAddr(dst,ii->eaddr) == 0)) {
dl_rpr->ldfd = open(filename, O_RDONLY, 0);
mopStartLoad(src, ii->eaddr, dl_rpr, trans);
- sprintf(line,
+ snprintf(line,sizeof(line),
"%x:%x:%x:%x:%x:%x Send me %s",
src[0],src[1],src[2],
src[3],src[4],src[5],pfile);
- syslog(LOG_INFO, line);
+ syslog(LOG_INFO, "%s", line);
}
}