summaryrefslogtreecommitdiff
path: root/bin
diff options
context:
space:
mode:
Diffstat (limited to 'bin')
-rw-r--r--bin/csh/csh.c31
-rw-r--r--bin/csh/csh.h6
-rw-r--r--bin/csh/dir.c28
-rw-r--r--bin/csh/dol.c18
-rw-r--r--bin/csh/exec.c35
-rw-r--r--bin/csh/extern.h4
-rw-r--r--bin/csh/file.c19
-rw-r--r--bin/csh/glob.c18
-rw-r--r--bin/csh/lex.c16
-rw-r--r--bin/csh/proc.c8
-rw-r--r--bin/csh/sem.c6
-rw-r--r--bin/csh/set.c8
-rw-r--r--bin/csh/str.c73
13 files changed, 175 insertions, 95 deletions
diff --git a/bin/csh/csh.c b/bin/csh/csh.c
index d0149a8f48f..0bdd06737bd 100644
--- a/bin/csh/csh.c
+++ b/bin/csh/csh.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: csh.c,v 1.18 2002/06/09 05:47:27 todd Exp $ */
+/* $OpenBSD: csh.c,v 1.19 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: csh.c,v 1.14 1995/04/29 23:21:28 mycroft Exp $ */
/*-
@@ -44,7 +44,7 @@ static char copyright[] =
#if 0
static char sccsid[] = "@(#)csh.c 8.2 (Berkeley) 10/12/93";
#else
-static char rcsid[] = "$OpenBSD: csh.c,v 1.18 2002/06/09 05:47:27 todd Exp $";
+static char rcsid[] = "$OpenBSD: csh.c,v 1.19 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -802,9 +802,9 @@ rechist()
*/
if ((shist = adrof(STRsavehist)) != NULL) {
if (shist->vec[0][0] != '\0')
- (void) Strcpy(hbuf, shist->vec[0]);
+ (void) Strlcpy(hbuf, shist->vec[0], sizeof hbuf/sizeof(Char));
else if ((shist = adrof(STRhistory)) && shist->vec[0][0] != '\0')
- (void) Strcpy(hbuf, shist->vec[0]);
+ (void) Strlcpy(hbuf, shist->vec[0], sizeof hbuf/sizeof(Char));
else
return;
}
@@ -812,8 +812,9 @@ rechist()
return;
if ((hfile = value(STRhistfile)) == STRNULL) {
- hfile = Strcpy(buf, value(STRhome));
- (void) Strcat(buf, STRsldthist);
+ Strlcpy(buf, value(STRhome), sizeof buf/sizeof(Char));
+ hfile = buf;
+ (void) Strlcat(buf, STRsldthist, sizeof buf/sizeof(Char));
}
if ((fp = open(short2str(hfile), O_WRONLY | O_CREAT | O_TRUNC,
@@ -1139,6 +1140,7 @@ dosource(v, t)
register Char *f;
bool hflg = 0;
Char buf[BUFSIZ];
+ char sbuf[BUFSIZ];
v++;
if (*v && eq(*v, STRmh)) {
@@ -1146,12 +1148,12 @@ dosource(v, t)
stderror(ERR_NAME | ERR_HFLAG);
hflg++;
}
- (void) Strcpy(buf, *v);
+ (void) Strlcpy(buf, *v, sizeof buf/sizeof(Char));
f = globone(buf, G_ERROR);
- (void) strcpy((char *) buf, short2str(f));
+ (void) strlcpy(sbuf, short2str(f), sizeof sbuf);
xfree((ptr_t) f);
- if (!srcfile((char *) buf, 0, hflg) && !hflg)
- stderror(ERR_SYSTEM, (char *) buf, strerror(errno));
+ if (!srcfile(sbuf, 0, hflg) && !hflg)
+ stderror(ERR_SYSTEM, sbuf, strerror(errno));
}
/*
@@ -1208,8 +1210,9 @@ mailchk()
* We write the home directory of the user back there.
*/
int
-gethdir(home)
+gethdir(home, len)
Char *home;
+ int len;
{
Char *h;
struct passwd *pw;
@@ -1219,7 +1222,8 @@ gethdir(home)
*/
if (*home == '\0') {
if ((h = value(STRhome)) != NULL) {
- (void) Strcpy(home, h);
+ if (Strlcpy(home, h, len) >= len)
+ return 1;
return 0;
}
else
@@ -1227,7 +1231,8 @@ gethdir(home)
}
if ((pw = getpwnam(short2str(home))) != NULL) {
- (void) Strcpy(home, str2short(pw->pw_dir));
+ if (Strlcpy(home, str2short(pw->pw_dir), len) >= len)
+ return 1;
return 0;
}
else
diff --git a/bin/csh/csh.h b/bin/csh/csh.h
index a7029b2ef38..4ec357dc518 100644
--- a/bin/csh/csh.h
+++ b/bin/csh/csh.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: csh.h,v 1.8 2002/02/16 21:27:06 millert Exp $ */
+/* $OpenBSD: csh.h,v 1.9 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: csh.h,v 1.9 1995/03/21 09:02:40 cgd Exp $ */
/*-
@@ -490,8 +490,10 @@ Char HISTSUB; /* auto-substitute character */
#define Strchr(a, b) strchr(a, b)
#define Strrchr(a, b) strrchr(a, b)
#define Strcat(a, b) strcat(a, b)
+#define Strlcat(a, b, l) strlcat(a, b, l)
#define Strncat(a, b, c) strncat(a, b, c)
#define Strcpy(a, b) strcpy(a, b)
+#define Strlcpy(a, b, l) strlcpy(a, b, l)
#define Strncpy(a, b, c) strncpy(a, b, c)
#define Strlen(a) strlen(a)
#define Strcmp(a, b) strcmp(a, b)
@@ -510,8 +512,10 @@ Char HISTSUB; /* auto-substitute character */
#define Strchr(a, b) s_strchr(a, b)
#define Strrchr(a, b) s_strrchr(a, b)
#define Strcat(a, b) s_strcat(a, b)
+#define Strlcat(a, b, l) s_strlcat(a, b, l)
#define Strncat(a, b, c) s_strncat(a, b, c)
#define Strcpy(a, b) s_strcpy(a, b)
+#define Strlcpy(a, b, l) s_strlcpy(a, b, l)
#define Strncpy(a, b, c) s_strncpy(a, b, c)
#define Strlen(a) s_strlen(a)
#define Strcmp(a, b) s_strcmp(a, b)
diff --git a/bin/csh/dir.c b/bin/csh/dir.c
index 64ad90d8754..1fd86717bbb 100644
--- a/bin/csh/dir.c
+++ b/bin/csh/dir.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: dir.c,v 1.8 2002/08/12 00:42:56 aaron Exp $ */
+/* $OpenBSD: dir.c,v 1.9 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: dir.c,v 1.9 1995/03/21 09:02:42 cgd Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)dir.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: dir.c,v 1.8 2002/08/12 00:42:56 aaron Exp $";
+static char rcsid[] = "$OpenBSD: dir.c,v 1.9 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -284,10 +284,11 @@ dnormalize(cp)
if (adrof(STRignore_symlinks)) {
int dotdot = 0;
Char *dp, *cwd;
+ size_t len;
- cwd = (Char *) xmalloc((size_t) ((Strlen(dcwd->di_name) + 3) *
- sizeof(Char)));
- (void) Strcpy(cwd, dcwd->di_name);
+ len = (size_t) (Strlen(dcwd->di_name) + 3);
+ cwd = (Char *) xmalloc(len * sizeof(Char));
+ (void) Strlcpy(cwd, dcwd->di_name, len);
/*
* Ignore . and count ..'s
@@ -474,7 +475,7 @@ dfollow(cp)
printd = 1;
return dgoto(cp);
}
- (void) strcpy(ebuf, short2str(cp));
+ (void) strlcpy(ebuf, short2str(cp), sizeof ebuf);
xfree((ptr_t) cp);
stderror(ERR_SYSTEM, ebuf, strerror(serrno));
return (NULL);
@@ -527,7 +528,7 @@ dopushd(v, t)
register Char *ccp;
ccp = dfollow(*v);
- dp = (struct directory *) xcalloc(sizeof(struct directory), 1);
+ dp = (struct directory *) xcalloc(1, sizeof(struct directory));
dp->di_name = ccp;
dp->di_count = 0;
dp->di_prev = dcwd;
@@ -657,9 +658,9 @@ dcanon(cp, p)
abort();
if (Strlen(p1) + Strlen(cp) + 1 >= MAXPATHLEN)
abort();
- (void) Strcpy(tmpdir, p1);
- (void) Strcat(tmpdir, STRslash);
- (void) Strcat(tmpdir, cp);
+ (void) Strlcpy(tmpdir, p1, sizeof tmpdir/sizeof(Char));
+ (void) Strlcat(tmpdir, STRslash, sizeof tmpdir/sizeof(Char));
+ (void) Strlcat(tmpdir, cp, sizeof tmpdir/sizeof(Char));
xfree((ptr_t) cp);
cp = p = Strsave(tmpdir);
}
@@ -705,7 +706,7 @@ dcanon(cp, p)
if (sp != cp && !adrof(STRignore_symlinks) &&
(cc = readlink(short2str(cp), tlink,
sizeof tlink-1)) >= 0) {
- (void) Strcpy(link, str2short(tlink));
+ (void) Strlcpy(link, str2short(tlink), sizeof link/sizeof(Char));
link[cc] = '\0';
if (slash)
@@ -790,7 +791,7 @@ dcanon(cp, p)
!adrof(STRignore_symlinks) &&
(cc = readlink(short2str(cp), tlink,
sizeof tlink-1)) >= 0) {
- (void) Strcpy(link, str2short(tlink));
+ (void) Strlcpy(link, str2short(tlink), sizeof link/sizeof(Char));
link[cc] = '\0';
/*
@@ -893,7 +894,8 @@ dcanon(cp, p)
/*
* Start comparing dev & ino backwards
*/
- p2 = Strcpy(link, cp);
+ Strlcpy(link, cp, sizeof link/sizeof(Char));
+ p2 = link;
for (sp = NULL; *p2 && stat(short2str(p2), &statbuf) != -1;) {
if (statbuf.st_dev == home_dev &&
statbuf.st_ino == home_ino) {
diff --git a/bin/csh/dol.c b/bin/csh/dol.c
index 00567ea7c39..9e215b220c6 100644
--- a/bin/csh/dol.c
+++ b/bin/csh/dol.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: dol.c,v 1.10 2002/06/09 05:47:27 todd Exp $ */
+/* $OpenBSD: dol.c,v 1.11 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: dol.c,v 1.8 1995/09/27 00:38:38 jtc Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)dol.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: dol.c,v 1.10 2002/06/09 05:47:27 todd Exp $";
+static char rcsid[] = "$OpenBSD: dol.c,v 1.11 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -470,7 +470,7 @@ Dgetdol()
break;
case '*':
- (void) Strcpy(name, STRargv);
+ (void) Strlcpy(name, STRargv, sizeof name/sizeof(Char));
vp = adrof(STRargv);
subscr = -1; /* Prevent eating [...] */
break;
@@ -729,12 +729,12 @@ setDolp(cp)
do {
dp = Strstr(cp, lhsub);
if (dp) {
- np = (Char *) xmalloc((size_t)
- ((Strlen(cp) + 1 - lhlen + rhlen) *
- sizeof(Char)));
- (void) Strncpy(np, cp, dp - cp);
- (void) Strcpy(np + (dp - cp), rhsub);
- (void) Strcpy(np + (dp - cp) + rhlen, dp + lhlen);
+ size_t len = Strlen(cp) + 1 - lhlen + rhlen;
+
+ np = (Char *) xmalloc(len * sizeof(Char));
+ (void) Strlcpy(np, cp, len);
+ (void) Strlcat(np, rhsub, len);
+ (void) Strlcat(np, dp + lhlen, len);
xfree((ptr_t) cp);
dp = cp = np;
diff --git a/bin/csh/exec.c b/bin/csh/exec.c
index f7da9108d48..b97bb1ad5f1 100644
--- a/bin/csh/exec.c
+++ b/bin/csh/exec.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: exec.c,v 1.10 2002/06/09 05:47:27 todd Exp $ */
+/* $OpenBSD: exec.c,v 1.11 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: exec.c,v 1.9 1996/09/30 20:03:54 christos Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)exec.c 8.3 (Berkeley) 5/23/95";
#else
-static char rcsid[] = "$OpenBSD: exec.c,v 1.10 2002/06/09 05:47:27 todd Exp $";
+static char rcsid[] = "$OpenBSD: exec.c,v 1.11 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -103,7 +103,7 @@ static Char *justabs[] = {STRNULL, 0};
static void pexerr(void);
static void texec(Char *, Char **);
static int hashname(Char *);
-static int tellmewhat(struct wordent *, Char *);
+static int tellmewhat(struct wordent *, Char *, int len);
static int executable(Char *, Char *, bool);
static int iscommand(Char *);
@@ -647,15 +647,16 @@ dowhich(v, c)
}
else {
lex[1].word = *v;
- set(STRstatus, Strsave(tellmewhat(lex, NULL) ? STR0 : STR1));
+ set(STRstatus, Strsave(tellmewhat(lex, NULL, 0) ? STR0 : STR1));
}
}
}
static int
-tellmewhat(lexp, str)
+tellmewhat(lexp, str, len)
struct wordent *lexp;
Char *str;
+ int len;
{
register int i;
register struct biltins *bptr;
@@ -700,13 +701,13 @@ tellmewhat(lexp, str)
for (bptr = bfunc; bptr < &bfunc[nbfunc]; bptr++) {
if (eq(sp->word, str2short(bptr->bname))) {
if (str == NULL) {
- if (aliased)
- prlex(cshout, lexp);
- (void) fprintf(cshout, "%s: shell built-in command.\n",
+ if (aliased)
+ prlex(cshout, lexp);
+ (void) fprintf(cshout, "%s: shell built-in command.\n",
vis_str(sp->word));
}
else
- (void) Strcpy(str, sp->word);
+ (void) Strlcpy(str, sp->word, len/sizeof(Char));
sp->word = s0; /* we save and then restore this */
return 1;
}
@@ -737,26 +738,26 @@ tellmewhat(lexp, str)
prlex(cshout, lexp);
}
else {
- s1 = Strspl(*pv, STRslash);
- sp->word = Strspl(s1, sp->word);
- xfree((ptr_t) s1);
+ s1 = Strspl(*pv, STRslash);
+ sp->word = Strspl(s1, sp->word);
+ xfree((ptr_t) s1);
if (str == NULL)
prlex(cshout, lexp);
else
- (void) Strcpy(str, sp->word);
- xfree((ptr_t) sp->word);
- }
+ (void) Strlcpy(str, sp->word, len/sizeof(Char));
+ xfree((ptr_t) sp->word);
+ }
found = 1;
}
else {
if (str == NULL) {
- if (aliased)
+ if (aliased)
prlex(cshout, lexp);
(void) fprintf(csherr,
"%s: Command not found.\n", vis_str(sp->word));
}
else
- (void) Strcpy(str, sp->word);
+ (void) Strlcpy(str, sp->word, len/sizeof(Char));
found = 0;
}
sp->word = s0; /* we save and then restore this */
diff --git a/bin/csh/extern.h b/bin/csh/extern.h
index 6b161488209..f5b7f7d4a48 100644
--- a/bin/csh/extern.h
+++ b/bin/csh/extern.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: extern.h,v 1.5 2002/02/17 19:42:18 millert Exp $ */
+/* $OpenBSD: extern.h,v 1.6 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: extern.h,v 1.8 1996/10/31 23:50:54 christos Exp $ */
/*-
@@ -41,7 +41,7 @@
/*
* csh.c
*/
-int gethdir(Char *);
+int gethdir(Char *, int);
void dosource(Char **, struct command *);
void exitstat(void);
void goodbye(void);
diff --git a/bin/csh/file.c b/bin/csh/file.c
index 422b2700fd1..585da7c9b71 100644
--- a/bin/csh/file.c
+++ b/bin/csh/file.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: file.c,v 1.12 2002/07/24 19:53:50 millert Exp $ */
+/* $OpenBSD: file.c,v 1.13 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: file.c,v 1.11 1996/11/08 19:34:37 christos Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)file.c 8.2 (Berkeley) 3/19/94";
#else
-static char rcsid[] = "$OpenBSD: file.c,v 1.12 2002/07/24 19:53:50 millert Exp $";
+static char rcsid[] = "$OpenBSD: file.c,v 1.13 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -222,7 +222,8 @@ filetype(dir, file)
Char path[MAXPATHLEN];
struct stat statb;
- catn(Strcpy(path, dir), file, sizeof(path) / sizeof(Char));
+ Strlcpy(path, dir, sizeof path/sizeof(Char));
+ catn(path, file, sizeof(path) / sizeof(Char));
if (lstat(short2str(path), &statb) == 0) {
switch (statb.st_mode & S_IFMT) {
case S_IFDIR:
@@ -301,21 +302,23 @@ tilde(new, old)
register struct passwd *pw;
static Char person[40];
- if (old[0] != '~')
- return (Strcpy(new, old));
+ if (old[0] != '~') {
+ Strlcpy(new, old, MAXPATHLEN);
+ return new;
+ }
for (p = person, o = &old[1]; *o && *o != '/'; *p++ = *o++)
continue;
*p = '\0';
if (person[0] == '\0')
- (void) Strcpy(new, value(STRhome));
+ (void) Strlcpy(new, value(STRhome), MAXPATHLEN);
else {
pw = getpwnam(short2str(person));
if (pw == NULL)
return (NULL);
- (void) Strcpy(new, str2short(pw->pw_dir));
+ (void) Strlcpy(new, str2short(pw->pw_dir), MAXPATHLEN);
}
- (void) Strcat(new, o);
+ (void) Strlcat(new, o, MAXPATHLEN);
return (new);
}
diff --git a/bin/csh/glob.c b/bin/csh/glob.c
index d012450b43f..0837a4a259d 100644
--- a/bin/csh/glob.c
+++ b/bin/csh/glob.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: glob.c,v 1.8 2002/06/09 05:47:27 todd Exp $ */
+/* $OpenBSD: glob.c,v 1.9 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: glob.c,v 1.10 1995/03/21 09:03:01 cgd Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)glob.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: glob.c,v 1.8 2002/06/09 05:47:27 todd Exp $";
+static char rcsid[] = "$OpenBSD: glob.c,v 1.9 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -110,7 +110,7 @@ globtilde(nv, s)
*b++ = *s++)
continue;
*b = EOS;
- if (gethdir(gstart)) {
+ if (gethdir(gstart, &gbuf[sizeof(gbuf)/sizeof(Char)] - gstart)) {
blkfree(nv);
if (*gstart)
stderror(ERR_UNKUSER, vis_str(gstart));
@@ -195,8 +195,8 @@ globbrace(s, p, bl)
Char savec = *pm;
*pm = EOS;
- (void) Strcpy(lm, pl);
- (void) Strcat(gbuf, pe + 1);
+ (void) Strlcpy(lm, pl, &gbuf[sizeof(gbuf)/sizeof(Char)] - lm);
+ (void) Strlcat(gbuf, pe + 1, MAXPATHLEN);
*pm = savec;
*vl++ = Strsave(gbuf);
len++;
@@ -371,7 +371,8 @@ handleone(str, vl, action)
str = Strspl(cp, *vlp);
xfree((ptr_t) cp);
}
- while (*++vlp);
+ while (*++vlp)
+ ;
blkfree(vl);
break;
case G_IGNORE:
@@ -422,7 +423,8 @@ libglob(vl)
}
gflgs |= GLOB_APPEND;
}
- while (*++vl);
+ while (*++vl)
+ ;
vl = (globv.gl_pathc == 0 || (magic && !match && !nonomatch)) ?
NULL : blk2short(globv.gl_pathv);
globfree(&globv);
@@ -933,7 +935,7 @@ sortscmp(a, b)
return (-1);
#if defined(NLS) && !defined(NOSTRCOLL)
- (void) strcpy(buf, short2str(*(Char **)a));
+ (void) strlcpy(buf, short2str(*(Char **)a), sizeof buf);
return ((int) strcoll(buf, short2str(*(Char **)b)));
#else
return ((int) Strcmp(*(Char **)a, *(Char **)b));
diff --git a/bin/csh/lex.c b/bin/csh/lex.c
index 84fbf766ee3..a87160b5f7a 100644
--- a/bin/csh/lex.c
+++ b/bin/csh/lex.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: lex.c,v 1.7 2002/06/09 05:47:27 todd Exp $ */
+/* $OpenBSD: lex.c,v 1.8 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: lex.c,v 1.9 1995/09/27 00:38:46 jtc Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)lex.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: lex.c,v 1.7 2002/06/09 05:47:27 todd Exp $";
+static char rcsid[] = "$OpenBSD: lex.c,v 1.8 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -635,10 +635,10 @@ addla(cp)
return;
}
if (lap)
- (void) Strcpy(buf, lap);
- (void) Strcpy(labuf, cp);
+ (void) Strlcpy(buf, lap, sizeof buf/sizeof(Char));
+ (void) Strlcpy(labuf, cp, sizeof labuf/sizeof(Char));
if (lap)
- (void) Strcat(labuf, buf);
+ (void) Strlcat(labuf, buf, sizeof labuf/sizeof(Char));
lap = labuf;
}
@@ -770,7 +770,7 @@ getsub(en)
seterror(ERR_NOSUBST);
return (en);
}
- (void) Strcpy(lhsb, slhs);
+ (void) Strlcpy(lhsb, slhs, sizeof(lhsb)/sizeof(Char));
break;
#ifdef notdef
@@ -816,7 +816,7 @@ getsub(en)
return (en);
}
cp = rhsb;
- (void) Strcpy(orhsb, cp);
+ (void) Strlcpy(orhsb, cp, sizeof(orhsb)/sizeof(Char));
for (;;) {
c = getC(0);
if (c == '\n') {
@@ -855,7 +855,7 @@ getsub(en)
seterror(ERR_BADBANGMOD, c);
return (en);
}
- (void) Strcpy(slhs, lhsb);
+ (void) Strlcpy(slhs, lhsb, sizeof(slhs)/sizeof(Char));
if (exclc)
en = dosub(sc, en, global);
}
diff --git a/bin/csh/proc.c b/bin/csh/proc.c
index e4dbfaa0d7c..4c69a71bc10 100644
--- a/bin/csh/proc.c
+++ b/bin/csh/proc.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: proc.c,v 1.15 2002/02/19 19:39:35 millert Exp $ */
+/* $OpenBSD: proc.c,v 1.16 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: proc.c,v 1.9 1995/04/29 23:21:33 mycroft Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)proc.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: proc.c,v 1.15 2002/02/19 19:39:35 millert Exp $";
+static char rcsid[] = "$OpenBSD: proc.c,v 1.16 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -600,12 +600,12 @@ pads(cp)
if (cmdlen >= PMAXLEN)
return;
if (cmdlen + i >= PMAXLEN) {
- (void) Strcpy(cmdp, STRsp3dots);
+ (void) Strlcpy(cmdp, STRsp3dots, PMAXLEN - i); /* XXX? */
cmdlen = PMAXLEN;
cmdp += 4;
return;
}
- (void) Strcpy(cmdp, cp);
+ (void) Strlcpy(cmdp, cp, PMAXLEN);
cmdp += i;
cmdlen += i;
}
diff --git a/bin/csh/sem.c b/bin/csh/sem.c
index 29e7bcd3165..d23ce8f68dc 100644
--- a/bin/csh/sem.c
+++ b/bin/csh/sem.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sem.c,v 1.11 2002/02/19 19:39:35 millert Exp $ */
+/* $OpenBSD: sem.c,v 1.12 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: sem.c,v 1.9 1995/09/27 00:38:50 jtc Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)sem.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: sem.c,v 1.11 2002/02/19 19:39:35 millert Exp $";
+static char rcsid[] = "$OpenBSD: sem.c,v 1.12 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -91,7 +91,7 @@ execute(t, wanttty, pipein, pipeout)
case NODE_COMMAND:
if ((t->t_dcom[0][0] & (QUOTE | TRIM)) == QUOTE)
- (void) Strcpy(t->t_dcom[0], t->t_dcom[0] + 1);
+ (void) Strcpy(t->t_dcom[0], t->t_dcom[0] + 1); /* safe */
if ((t->t_dflg & F_REPEAT) == 0)
Dfix(t); /* $ " ' \ */
if (t->t_dcom[0] == 0)
diff --git a/bin/csh/set.c b/bin/csh/set.c
index 8d04fa3bf45..ad6864c79f9 100644
--- a/bin/csh/set.c
+++ b/bin/csh/set.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: set.c,v 1.7 2002/02/19 19:39:35 millert Exp $ */
+/* $OpenBSD: set.c,v 1.8 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: set.c,v 1.8 1995/03/21 18:35:52 mycroft Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)set.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: set.c,v 1.7 2002/02/19 19:39:35 millert Exp $";
+static char rcsid[] = "$OpenBSD: set.c,v 1.8 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -665,10 +665,10 @@ exportpath(val)
"Warning: ridiculously long PATH truncated\n");
break;
}
- (void) Strcat(exppath, *val++);
+ (void) Strlcat(exppath, *val++, sizeof exppath/sizeof(Char));
if (*val == 0 || eq(*val, STRRparen))
break;
- (void) Strcat(exppath, STRcolon);
+ (void) Strlcat(exppath, STRcolon, sizeof exppath/sizeof(Char));
}
Setenv(STRPATH, exppath);
}
diff --git a/bin/csh/str.c b/bin/csh/str.c
index 7bbc8bf5ca3..bab76c9c89a 100644
--- a/bin/csh/str.c
+++ b/bin/csh/str.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: str.c,v 1.6 2002/06/09 05:47:27 todd Exp $ */
+/* $OpenBSD: str.c,v 1.7 2003/01/08 06:54:16 deraadt Exp $ */
/* $NetBSD: str.c,v 1.6 1995/03/21 09:03:24 cgd Exp $ */
/*-
@@ -38,7 +38,7 @@
#if 0
static char sccsid[] = "@(#)str.c 8.1 (Berkeley) 5/31/93";
#else
-static char rcsid[] = "$OpenBSD: str.c,v 1.6 2002/06/09 05:47:27 todd Exp $";
+static char rcsid[] = "$OpenBSD: str.c,v 1.7 2003/01/08 06:54:16 deraadt Exp $";
#endif
#endif /* not lint */
@@ -173,6 +173,66 @@ s_strcpy(dst, src)
return (sdst);
}
+size_t
+s_strlcpy(dst, src, siz)
+ Char *dst;
+ const Char *src;
+ size_t siz;
+{
+ register Char *d = dst;
+ register const Char *s = src;
+ register size_t n = siz;
+
+ /* Copy as many bytes as will fit */
+ if (n != 0 && --n != 0) {
+ do {
+ if ((*d++ = *s++) == 0)
+ break;
+ } while (--n != 0);
+ }
+
+ /* Not enough room in dst, add NUL and traverse rest of src */
+ if (n == 0) {
+ if (siz != 0)
+ *d = '\0'; /* NUL-terminate dst */
+ while (*s++)
+ ;
+ }
+
+ return(s - src - 1); /* count does not include NUL */
+}
+
+size_t
+s_strlcat(dst, src, siz)
+ Char *dst;
+ const Char *src;
+ size_t siz;
+{
+ register Char *d = dst;
+ register const Char *s = src;
+ register size_t n = siz;
+ size_t dlen;
+
+ /* Find the end of dst and adjust bytes left but don't go past end */
+ while (n-- != 0 && *d != '\0')
+ d++;
+ dlen = d - dst;
+ n = siz - dlen;
+
+ if (n == 0)
+ return(dlen + s_strlen((Char *)s));
+ while (*s != '\0') {
+ if (n != 1) {
+ *d++ = *s;
+ n--;
+ }
+ s++;
+ }
+ *d = '\0';
+
+ return(dlen + (s - src)); /* count does not include NUL */
+}
+
Char *
s_strncpy(dst, src, n)
register Char *dst, *src;
@@ -190,7 +250,8 @@ s_strncpy(dst, src, n)
*dst++ = '\0';
return(sdst);
}
- while (--n != 0);
+ while (--n != 0)
+ ;
return (sdst);
}
@@ -246,7 +307,8 @@ s_strchr(str, ch)
do
if (*str == ch)
return (str);
- while (*str++);
+ while (*str++)
+ ;
return (NULL);
}
@@ -261,7 +323,8 @@ s_strrchr(str, ch)
do
if (*str == ch)
rstr = str;
- while (*str++);
+ while (*str++)
+ ;
return (rstr);
}