summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--sys/kern/exec_script.c8
-rw-r--r--sys/kern/kern_malloc.c21
-rw-r--r--sys/kern/kern_sig.c4
-rw-r--r--sys/kern/kern_sysctl.c5
-rw-r--r--sys/kern/subr_autoconf.c4
-rw-r--r--sys/kern/subr_userconf.c4
-rw-r--r--sys/net/if_bridge.c4
-rw-r--r--sys/net/if_enc.c4
-rw-r--r--sys/net/if_faith.c4
-rw-r--r--sys/net/if_gif.c5
-rw-r--r--sys/net/if_loop.c4
-rw-r--r--sys/net/if_pflog.c4
-rw-r--r--sys/net/if_pfsync.c4
-rw-r--r--sys/net/if_ppp.c4
-rw-r--r--sys/net/if_sl.c5
-rw-r--r--sys/net/if_strip.c5
-rw-r--r--sys/net/if_tun.c4
-rw-r--r--sys/net/if_vlan.c4
18 files changed, 53 insertions, 44 deletions
diff --git a/sys/kern/exec_script.c b/sys/kern/exec_script.c
index 1d6ce8ed3c9..702ea6345ac 100644
--- a/sys/kern/exec_script.c
+++ b/sys/kern/exec_script.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: exec_script.c,v 1.17 2002/09/18 22:01:15 art Exp $ */
+/* $OpenBSD: exec_script.c,v 1.18 2003/05/03 21:14:59 deraadt Exp $ */
/* $NetBSD: exec_script.c,v 1.13 1996/02/04 02:15:06 christos Exp $ */
/*
@@ -203,10 +203,10 @@ check_shell:
MALLOC(shellargp, char **, 4 * sizeof(char *), M_EXEC, M_WAITOK);
tmpsap = shellargp;
*tmpsap = malloc(shellnamelen + 1, M_EXEC, M_WAITOK);
- strcpy(*tmpsap++, shellname);
+ strlcpy(*tmpsap++, shellname, shellnamelen + 1);
if (shellarg != NULL) {
*tmpsap = malloc(shellarglen + 1, M_EXEC, M_WAITOK);
- strcpy(*tmpsap++, shellarg);
+ strlcpy(*tmpsap++, shellarg, shellarglen + 1);
}
*tmpsap = malloc(MAXPATHLEN, M_EXEC, M_WAITOK);
#ifdef FDSCRIPTS
@@ -221,7 +221,7 @@ check_shell:
#endif
#ifdef FDSCRIPTS
} else
- sprintf(*tmpsap++, "/dev/fd/%d", epp->ep_fd);
+ snprintf(*tmpsap++, MAXPATHLEN, "/dev/fd/%d", epp->ep_fd);
#endif
*tmpsap = NULL;
diff --git a/sys/kern/kern_malloc.c b/sys/kern/kern_malloc.c
index f72652b8f24..91a7ad9e0a3 100644
--- a/sys/kern/kern_malloc.c
+++ b/sys/kern/kern_malloc.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kern_malloc.c,v 1.50 2003/04/10 08:48:34 tedu Exp $ */
+/* $OpenBSD: kern_malloc.c,v 1.51 2003/05/03 21:14:59 deraadt Exp $ */
/* $NetBSD: kern_malloc.c,v 1.15.4.2 1996/06/13 17:10:56 cgd Exp $ */
/*
@@ -536,9 +536,12 @@ sysctl_malloc(name, namelen, oldp, oldlenp, newp, newlen, p)
if (buckstring_init == 0) {
buckstring_init = 1;
bzero(buckstring, sizeof(buckstring));
- for (siz = 0, i = MINBUCKET; i < MINBUCKET + 16; i++)
- siz += sprintf(buckstring + siz,
- "%d,", (u_int)(1<<i));
+ for (siz = 0, i = MINBUCKET; i < MINBUCKET + 16; i++) {
+ snprintf(buckstring + siz,
+ sizeof buckstring - siz,
+ "%d,", (u_int)(1<<i));
+ siz += strlen(buckstring + siz);
+ }
/* Remove trailing comma */
if (siz)
buckstring[siz - 1] = '\0';
@@ -575,10 +578,12 @@ sysctl_malloc(name, namelen, oldp, oldlenp, newp, newlen, p)
}
memall = malloc(totlen + M_LAST, M_SYSCTL, M_WAITOK);
bzero(memall, totlen + M_LAST);
- for (siz = 0, i = 0; i < M_LAST; i++)
- siz += sprintf(memall + siz, "%s,",
- memname[i] ? memname[i] : "");
-
+ for (siz = 0, i = 0; i < M_LAST; i++) {
+ snprintf(memall + siz,
+ totlen + M_LAST - siz,
+ "%s,", memname[i] ? memname[i] : "");
+ siz += strlen(memall + siz);
+ }
/* Remove trailing comma */
if (siz)
memall[siz - 1] = '\0';
diff --git a/sys/kern/kern_sig.c b/sys/kern/kern_sig.c
index 83a27b631c7..df8abc28e3d 100644
--- a/sys/kern/kern_sig.c
+++ b/sys/kern/kern_sig.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kern_sig.c,v 1.61 2002/10/01 17:33:39 art Exp $ */
+/* $OpenBSD: kern_sig.c,v 1.62 2003/05/03 21:14:59 deraadt Exp $ */
/* $NetBSD: kern_sig.c,v 1.54 1996/04/22 01:38:32 christos Exp $ */
/*
@@ -1309,7 +1309,7 @@ coredump(p)
cred->cr_uid = p->p_cred->p_ruid;
cred->cr_gid = p->p_cred->p_rgid;
- sprintf(name, "%s.core", p->p_comm);
+ snprintf(name, sizeof name, "%s.core", p->p_comm);
NDINIT(&nd, LOOKUP, NOFOLLOW, UIO_SYSSPACE, name, p);
error = vn_open(&nd, O_CREAT | FWRITE | O_NOFOLLOW, S_IRUSR | S_IWUSR);
diff --git a/sys/kern/kern_sysctl.c b/sys/kern/kern_sysctl.c
index 338027df657..62a58eeaea2 100644
--- a/sys/kern/kern_sysctl.c
+++ b/sys/kern/kern_sysctl.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kern_sysctl.c,v 1.80 2003/04/25 20:06:41 grange Exp $ */
+/* $OpenBSD: kern_sysctl.c,v 1.81 2003/05/03 21:14:59 deraadt Exp $ */
/* $NetBSD: kern_sysctl.c,v 1.17 1996/05/20 17:49:05 mrg Exp $ */
/*-
@@ -1284,8 +1284,9 @@ sysctl_diskinit(update, p)
for (dk = TAILQ_FIRST(&disklist), i = 0, l = 0; dk;
dk = TAILQ_NEXT(dk, dk_link), i++) {
- l += sprintf(disknames + l, "%s,",
+ snprintf(disknames + l, tlen - l, "%s,",
dk->dk_name ? dk->dk_name : "");
+ l += strlen(disknames + l);
sdk = diskstats + i;
sdk->ds_busy = dk->dk_busy;
sdk->ds_xfer = dk->dk_xfer;
diff --git a/sys/kern/subr_autoconf.c b/sys/kern/subr_autoconf.c
index 1ad221818b5..9cfacee759a 100644
--- a/sys/kern/subr_autoconf.c
+++ b/sys/kern/subr_autoconf.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: subr_autoconf.c,v 1.34 2003/04/19 19:08:53 krw Exp $ */
+/* $OpenBSD: subr_autoconf.c,v 1.35 2003/05/03 21:14:59 deraadt Exp $ */
/* $NetBSD: subr_autoconf.c,v 1.21 1996/04/04 06:06:18 cgd Exp $ */
/*
@@ -848,6 +848,6 @@ evcnt_attach(dev, name, ev)
/* ev->ev_next = NULL; */
ev->ev_dev = dev;
/* ev->ev_count = 0; */
- strcpy(ev->ev_name, name);
+ strlcpy(ev->ev_name, name, sizeof ev->ev_name);
TAILQ_INSERT_TAIL(&allevents, ev, ev_list);
}
diff --git a/sys/kern/subr_userconf.c b/sys/kern/subr_userconf.c
index cc8868ef18d..613bfe742a2 100644
--- a/sys/kern/subr_userconf.c
+++ b/sys/kern/subr_userconf.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: subr_userconf.c,v 1.31 2002/07/29 23:18:57 art Exp $ */
+/* $OpenBSD: subr_userconf.c,v 1.32 2003/05/03 21:14:59 deraadt Exp $ */
/*
* Copyright (c) 1996-2001 Mats O Jansson <moj@stacken.kth.se>
@@ -183,7 +183,7 @@ void
userconf_hist_int(val)
int val;
{
- sprintf(userconf_histbuf," %d",val);
+ snprintf(userconf_histbuf, sizeof userconf_histbuf, " %d",val);
if (userconf_histcur + strlen(userconf_histbuf) < userconf_histsz) {
bcopy(userconf_histbuf,
&userconf_history[userconf_histcur],
diff --git a/sys/net/if_bridge.c b/sys/net/if_bridge.c
index 20def188a2c..b6a2aa0382a 100644
--- a/sys/net/if_bridge.c
+++ b/sys/net/if_bridge.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_bridge.c,v 1.114 2003/03/31 22:59:47 millert Exp $ */
+/* $OpenBSD: if_bridge.c,v 1.115 2003/05/03 21:15:11 deraadt Exp $ */
/*
* Copyright (c) 1999, 2000 Jason L. Wright (jason@thought.net)
@@ -206,7 +206,7 @@ bridgeattach(n)
LIST_INIT(&sc->sc_iflist);
LIST_INIT(&sc->sc_spanlist);
ifp = &sc->sc_if;
- sprintf(ifp->if_xname, "bridge%d", i);
+ snprintf(ifp->if_xname, sizeof ifp->if_xname, "bridge%d", i);
ifp->if_softc = sc;
ifp->if_mtu = ETHERMTU;
ifp->if_ioctl = bridge_ioctl;
diff --git a/sys/net/if_enc.c b/sys/net/if_enc.c
index 8411f35eff5..4a7cc72a5c8 100644
--- a/sys/net/if_enc.c
+++ b/sys/net/if_enc.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_enc.c,v 1.39 2002/06/30 13:04:36 itojun Exp $ */
+/* $OpenBSD: if_enc.c,v 1.40 2003/05/03 21:15:11 deraadt Exp $ */
/*
* The authors of this code are John Ioannidis (ji@tla.org),
* Angelos D. Keromytis (kermit@csd.uch.gr) and
@@ -101,7 +101,7 @@ encattach(int nenc)
for (i = 0; i < NENC; i++)
{
ifp = &encif[i].sc_if;
- sprintf(ifp->if_xname, "enc%d", i);
+ snprintf(ifp->if_xname, sizeof ifp->if_xname, "enc%d", i);
ifp->if_softc = &encif[i];
ifp->if_mtu = ENCMTU;
ifp->if_ioctl = encioctl;
diff --git a/sys/net/if_faith.c b/sys/net/if_faith.c
index 45d6d7a5192..32ab43db223 100644
--- a/sys/net/if_faith.c
+++ b/sys/net/if_faith.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_faith.c,v 1.13 2003/05/03 01:43:07 itojun Exp $ */
+/* $OpenBSD: if_faith.c,v 1.14 2003/05/03 21:15:11 deraadt Exp $ */
/*
* Copyright (c) 1982, 1986, 1993
* The Regents of the University of California. All rights reserved.
@@ -92,7 +92,7 @@ faithattach(faith)
for (i = 0; i < NFAITH; i++) {
ifp = &faithif[i];
bzero(ifp, sizeof(faithif[i]));
- sprintf(ifp->if_xname, "faith%d", i);
+ snprintf(ifp->if_xname, sizeof ifp->if_xname, "faith%d", i);
ifp->if_mtu = FAITHMTU;
/* Change to BROADCAST experimentaly to announce its prefix. */
ifp->if_flags = /* IFF_LOOPBACK */ IFF_BROADCAST | IFF_MULTICAST;
diff --git a/sys/net/if_gif.c b/sys/net/if_gif.c
index d7bc71ed518..d790dcb256d 100644
--- a/sys/net/if_gif.c
+++ b/sys/net/if_gif.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_gif.c,v 1.27 2003/05/03 01:43:07 itojun Exp $ */
+/* $OpenBSD: if_gif.c,v 1.28 2003/05/03 21:15:11 deraadt Exp $ */
/* $KAME: if_gif.c,v 1.43 2001/02/20 08:51:07 itojun Exp $ */
/*
@@ -83,7 +83,8 @@ gifattach(n)
M_DEVBUF, M_WAIT);
bzero(sc, ngif * sizeof(struct gif_softc));
for (i = 0; i < ngif; sc++, i++) {
- sprintf(sc->gif_if.if_xname, "gif%d", i);
+ snprintf(sc->gif_if.if_xname, sizeof sc->gif_if.if_xname,
+ "gif%d", i);
sc->gif_if.if_mtu = GIF_MTU;
sc->gif_if.if_flags = IFF_POINTOPOINT | IFF_MULTICAST;
sc->gif_if.if_ioctl = gif_ioctl;
diff --git a/sys/net/if_loop.c b/sys/net/if_loop.c
index 76a971badb6..6247c21ca84 100644
--- a/sys/net/if_loop.c
+++ b/sys/net/if_loop.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_loop.c,v 1.27 2003/05/03 01:43:07 itojun Exp $ */
+/* $OpenBSD: if_loop.c,v 1.28 2003/05/03 21:15:11 deraadt Exp $ */
/* $NetBSD: if_loop.c,v 1.15 1996/05/07 02:40:33 thorpej Exp $ */
/*
@@ -191,7 +191,7 @@ loopattach(n)
bzero(ifp, sizeof(struct ifnet));
if (i == 0)
lo0ifp = ifp;
- sprintf(ifp->if_xname, "lo%d", i);
+ snprintf(ifp->if_xname, sizeof ifp->if_xname, "lo%d", i);
ifp->if_softc = NULL;
ifp->if_mtu = LOMTU;
ifp->if_flags = IFF_LOOPBACK | IFF_MULTICAST;
diff --git a/sys/net/if_pflog.c b/sys/net/if_pflog.c
index 7c1648ce6e5..3c93560d329 100644
--- a/sys/net/if_pflog.c
+++ b/sys/net/if_pflog.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_pflog.c,v 1.7 2002/10/29 19:51:04 mickey Exp $ */
+/* $OpenBSD: if_pflog.c,v 1.8 2003/05/03 21:15:11 deraadt Exp $ */
/*
* The authors of this code are John Ioannidis (ji@tla.org),
* Angelos D. Keromytis (kermit@csd.uch.gr) and
@@ -93,7 +93,7 @@ pflogattach(int npflog)
for (i = 0; i < NPFLOG; i++) {
ifp = &pflogif[i].sc_if;
- sprintf(ifp->if_xname, "pflog%d", i);
+ snprintf(ifp->if_xname, sizeof ifp->if_xname, "pflog%d", i);
ifp->if_softc = &pflogif[i];
ifp->if_mtu = PFLOGMTU;
ifp->if_ioctl = pflogioctl;
diff --git a/sys/net/if_pfsync.c b/sys/net/if_pfsync.c
index a14bd82f3b3..89a4f223ad9 100644
--- a/sys/net/if_pfsync.c
+++ b/sys/net/if_pfsync.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_pfsync.c,v 1.4 2002/12/23 18:53:59 mickey Exp $ */
+/* $OpenBSD: if_pfsync.c,v 1.5 2003/05/03 21:15:11 deraadt Exp $ */
/*
* Copyright (c) 2002 Michael Shalayeff
@@ -91,7 +91,7 @@ pfsyncattach(int npfsync)
pfsyncif.sc_ptr = NULL;
pfsyncif.sc_count = 8;
ifp = &pfsyncif.sc_if;
- strcpy(ifp->if_xname, "pfsync0");
+ strlcpy(ifp->if_xname, "pfsync0", sizeof ifp->if_xname);
ifp->if_softc = &pfsyncif;
ifp->if_ioctl = pfsyncioctl;
ifp->if_output = pfsyncoutput;
diff --git a/sys/net/if_ppp.c b/sys/net/if_ppp.c
index 738535168dd..563a94bcd3a 100644
--- a/sys/net/if_ppp.c
+++ b/sys/net/if_ppp.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_ppp.c,v 1.33 2003/02/12 14:41:07 jason Exp $ */
+/* $OpenBSD: if_ppp.c,v 1.34 2003/05/03 21:15:11 deraadt Exp $ */
/* $NetBSD: if_ppp.c,v 1.39 1997/05/17 21:11:59 christos Exp $ */
/*
@@ -229,7 +229,7 @@ pppattach()
for (sc = ppp_softc; i < NPPP; sc++) {
sc->sc_unit = i; /* XXX */
- sprintf(sc->sc_if.if_xname, "ppp%d", i++);
+ snprintf(sc->sc_if.if_xname, sizeof sc->sc_if.if_xname, "ppp%d", i++);
sc->sc_if.if_softc = sc;
sc->sc_if.if_mtu = PPP_MTU;
sc->sc_if.if_flags = IFF_POINTOPOINT | IFF_MULTICAST;
diff --git a/sys/net/if_sl.c b/sys/net/if_sl.c
index d9b16826e61..a43d99e47e4 100644
--- a/sys/net/if_sl.c
+++ b/sys/net/if_sl.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_sl.c,v 1.18 2003/01/07 09:00:33 kjc Exp $ */
+/* $OpenBSD: if_sl.c,v 1.19 2003/05/03 21:15:11 deraadt Exp $ */
/* $NetBSD: if_sl.c,v 1.39.4.1 1996/06/02 16:26:31 thorpej Exp $ */
/*
@@ -206,7 +206,8 @@ slattach(n)
bzero(sl_softc, n * sizeof(struct sl_softc));
for (sc = sl_softc; i < nsl; sc++) {
sc->sc_unit = i; /* XXX */
- sprintf(sc->sc_if.if_xname, "sl%d", i++);
+ snprintf(sc->sc_if.if_xname, sizeof sc->sc_if.if_xname,
+ "sl%d", i++);
sc->sc_if.if_softc = sc;
sc->sc_if.if_mtu = SLMTU;
sc->sc_if.if_flags =
diff --git a/sys/net/if_strip.c b/sys/net/if_strip.c
index f9b31036eba..1db330d1caa 100644
--- a/sys/net/if_strip.c
+++ b/sys/net/if_strip.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_strip.c,v 1.23 2003/01/07 09:00:33 kjc Exp $ */
+/* $OpenBSD: if_strip.c,v 1.24 2003/05/03 21:15:11 deraadt Exp $ */
/* $NetBSD: if_strip.c,v 1.2.4.3 1996/08/03 00:58:32 jtc Exp $ */
/* from: NetBSD: if_sl.c,v 1.38 1996/02/13 22:00:23 christos Exp $ */
@@ -351,7 +351,8 @@ stripattach(n)
for (sc = st_softc; i < NSTRIP; sc++) {
timeout_set(&sc->sc_timo, strip_timeout, sc);
sc->sc_unit = i; /* XXX */
- sprintf(sc->sc_if.if_xname, "strip%d", i++);
+ snprintf(sc->sc_if.if_xname, sizeof sc->sc_if.if_xname,
+ "strip%d", i++);
sc->sc_if.if_softc = sc;
sc->sc_if.if_mtu = SLMTU;
sc->sc_if.if_flags = 0;
diff --git a/sys/net/if_tun.c b/sys/net/if_tun.c
index 62046c91162..a0cfcc85016 100644
--- a/sys/net/if_tun.c
+++ b/sys/net/if_tun.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_tun.c,v 1.46 2003/04/18 05:26:13 jason Exp $ */
+/* $OpenBSD: if_tun.c,v 1.47 2003/05/03 21:15:11 deraadt Exp $ */
/* $NetBSD: if_tun.c,v 1.24 1996/05/07 02:40:48 thorpej Exp $ */
/*
@@ -151,7 +151,7 @@ tunattach(n)
tunctl[i].tun_flags = TUN_INITED;
ifp = &tunctl[i].tun_if;
- sprintf(ifp->if_xname, "tun%d", i);
+ snprintf(ifp->if_xname, sizeof ifp->if_xname, "tun%d", i);
ifp->if_softc = &tunctl[i];
ifp->if_mtu = TUNMTU;
ifp->if_ioctl = tun_ioctl;
diff --git a/sys/net/if_vlan.c b/sys/net/if_vlan.c
index 22ee8975afe..5fcec917897 100644
--- a/sys/net/if_vlan.c
+++ b/sys/net/if_vlan.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_vlan.c,v 1.36 2003/05/03 01:43:07 itojun Exp $ */
+/* $OpenBSD: if_vlan.c,v 1.37 2003/05/03 21:15:11 deraadt Exp $ */
/*
* Copyright 1998 Massachusetts Institute of Technology
*
@@ -113,7 +113,7 @@ vlanattach(int count)
LIST_INIT(&ifv_softc[i].vlan_mc_listhead);
ifp = &ifv_softc[i].ifv_if;
ifp->if_softc = &ifv_softc[i];
- sprintf(ifp->if_xname, "vlan%d", i);
+ snprintf(ifp->if_xname, sizeof ifp->if_xname, "vlan%d", i);
/* NB: flags are not set here */
/* NB: mtu is not set here */