aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjoerg <joerg@FreeBSD.org>1998-06-28 16:04:07 +0800
committerjoerg <joerg@FreeBSD.org>1998-06-28 16:04:07 +0800
commitbeb8a89d2d0257bef51f21b17253fa9775c0981e (patch)
tree8ab26aa90136eafcf1c2582d563d25c5996ff2f0
parentf464eed4526d08ff3026c44b464b7432c2f0c55b (diff)
downloadfreebsd-ports-gnome-beb8a89d2d0257bef51f21b17253fa9775c0981e.tar.gz
freebsd-ports-gnome-beb8a89d2d0257bef51f21b17253fa9775c0981e.tar.zst
freebsd-ports-gnome-beb8a89d2d0257bef51f21b17253fa9775c0981e.zip
Well, it's always a bad idea to introduce new bugs when trying to fix
one... strncat(3) takes a string as its second arg, and a len as its third one, not the other way round. =:-) The ``consistency'' of the C library sucks, of course.
-rw-r--r--mail/popper/files/patch-ag2
1 files changed, 1 insertions, 1 deletions
diff --git a/mail/popper/files/patch-ag b/mail/popper/files/patch-ag
index 9ed0df9ccc06..b86319be3e93 100644
--- a/mail/popper/files/patch-ag
+++ b/mail/popper/files/patch-ag
@@ -45,7 +45,7 @@
/* Append the <CR><LF> */
- (void)strcat(message, "\r\n");
+ len -= strlen(message);
-+ (void)strncat(message, len, "\r\n");
++ (void)strncat(message, "\r\n", len);
/* Send the message to the client */
(void)fputs(message,p->output);