diff options
author | 2001-05-16 12:48:31 +0000 | |
---|---|---|
committer | 2001-05-16 12:48:31 +0000 | |
commit | d424e204e60bfd7ffd01a57d3983f326a20fef28 (patch) | |
tree | 49e17bdb4138ce9da9ed97cdb6b57b088efe1497 | |
parent | document SMALL_KERNEL. (diff) | |
download | wireguard-openbsd-d424e204e60bfd7ffd01a57d3983f326a20fef28.tar.xz wireguard-openbsd-d424e204e60bfd7ffd01a57d3983f326a20fef28.zip |
No need to check M_WAIT/M_WAITOK malloc return values. (art@ ok)
67 files changed, 146 insertions, 404 deletions
diff --git a/sys/adosfs/advfsops.c b/sys/adosfs/advfsops.c index f4508f7a029..93c61772029 100644 --- a/sys/adosfs/advfsops.c +++ b/sys/adosfs/advfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: advfsops.c,v 1.18 2001/02/20 01:50:08 assar Exp $ */ +/* $OpenBSD: advfsops.c,v 1.19 2001/05/16 12:48:31 ho Exp $ */ /* $NetBSD: advfsops.c,v 1.24 1996/12/22 10:10:12 cgd Exp $ */ /* @@ -243,8 +243,7 @@ adosfs_mountfs(devvp, mp, p) /* allocate and load bitmap, set free space */ amp->bitmap = malloc(((amp->numblks + 31) / 32) * sizeof(*amp->bitmap), M_ADOSFSBITMAP, M_WAITOK); - if (amp->bitmap) - adosfs_loadbitmap(amp); + adosfs_loadbitmap(amp); if (mp->mnt_flag & MNT_RDONLY && amp->bitmap) { /* * Don't need the bitmap any more if it's read-only. diff --git a/sys/arch/amiga/dev/view.c b/sys/arch/amiga/dev/view.c index 83cba6231c5..bc29ae7b157 100644 --- a/sys/arch/amiga/dev/view.c +++ b/sys/arch/amiga/dev/view.c @@ -1,4 +1,4 @@ -/* $OpenBSD: view.c,v 1.3 1997/01/16 09:25:30 niklas Exp $ */ +/* $OpenBSD: view.c,v 1.4 2001/05/16 12:49:44 ho Exp $ */ /* $NetBSD: view.c,v 1.16 1996/10/13 03:07:35 christos Exp $ */ /* @@ -350,8 +350,6 @@ view_get_colormap (vu, ucm) /* add one incase of zero, ick. */ cme = malloc(sizeof (u_long)*(ucm->size + 1), M_IOCTLOPS, M_WAITOK); - if (cme == NULL) - return(ENOMEM); uep = ucm->entry; error = 0; @@ -376,8 +374,6 @@ view_set_colormap(vu, ucm) error = 0; cm = malloc(sizeof(u_long) * ucm->size + sizeof (*cm), M_IOCTLOPS, M_WAITOK); - if (cm == NULL) - return(ENOMEM); bcopy (ucm, cm, sizeof(colormap_t)); cm->entry = (u_long *)&cm[1]; /* table directly after. */ diff --git a/sys/arch/i386/isa/pcvt/pcvt_ext.c b/sys/arch/i386/isa/pcvt/pcvt_ext.c index cce3eed5ca1..795ef3def75 100644 --- a/sys/arch/i386/isa/pcvt/pcvt_ext.c +++ b/sys/arch/i386/isa/pcvt/pcvt_ext.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pcvt_ext.c,v 1.30 2001/05/16 05:07:48 millert Exp $ */ +/* $OpenBSD: pcvt_ext.c,v 1.31 2001/05/16 12:49:45 ho Exp $ */ /* * Copyright (c) 1992, 1995 Hellmuth Michaelis and Joerg Wunsch. * @@ -1125,16 +1125,11 @@ wd90c11_col(int cols) { if(vgacs[i].loaded) { - if((sv_fontwd[i] = - (u_char *)malloc(32 * 256, - M_DEVBUF, - M_WAITOK)) - == NULL) - printf("pcvt: no font buffer\n"); - else - vga_move_charset(i, - sv_fontwd[i], - 1); + sv_fontwd[i] = + (u_char *)malloc(32 * 256, + M_DEVBUF, + M_WAITOK); + vga_move_charset(i, sv_fontwd[i], 1); } else { diff --git a/sys/arch/i386/isa/pcvt/pcvt_kbd.c b/sys/arch/i386/isa/pcvt/pcvt_kbd.c index deb157f6bc2..c570dc29ea4 100644 --- a/sys/arch/i386/isa/pcvt/pcvt_kbd.c +++ b/sys/arch/i386/isa/pcvt/pcvt_kbd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pcvt_kbd.c,v 1.40 2001/01/22 18:48:43 deraadt Exp $ */ +/* $OpenBSD: pcvt_kbd.c,v 1.41 2001/05/16 12:49:45 ho Exp $ */ /* * Copyright (c) 1992, 1995 Hellmuth Michaelis and Joerg Wunsch. @@ -629,10 +629,9 @@ void ovlinit(int force) int i; if (force || ovlinitflag == 0) { - if (ovlinitflag == 0 && - (ovltbl = (Ovl_tbl *)malloc(sizeof(Ovl_tbl) * OVLTBL_SIZE, - M_DEVBUF, M_WAITOK)) == NULL) - panic("pcvt_kbd: malloc of Ovl_tbl failed"); + if (ovlinitflag == 0) + ovltbl = (Ovl_tbl *)malloc(sizeof(Ovl_tbl) * OVLTBL_SIZE, + M_DEVBUF, M_WAITOK); for(i = 0; i < OVLTBL_SIZE; i++) { ovltbl[i].keynum = diff --git a/sys/arch/i386/isa/pcvt/pcvt_out.c b/sys/arch/i386/isa/pcvt/pcvt_out.c index 8a8a23cfc80..d4a9556a7f0 100644 --- a/sys/arch/i386/isa/pcvt/pcvt_out.c +++ b/sys/arch/i386/isa/pcvt/pcvt_out.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pcvt_out.c,v 1.30 2001/01/22 18:48:44 deraadt Exp $ */ +/* $OpenBSD: pcvt_out.c,v 1.31 2001/05/16 12:49:45 ho Exp $ */ /* * Copyright (c) 1992, 1995 Hellmuth Michaelis and Joerg Wunsch. @@ -1261,12 +1261,9 @@ vt_coldmalloc(void) } scrollback_pages = SCROLLBACK_PAGES; - if ((Scrollbuffer = (u_short *)malloc(vs[0].maxcol * - vs[0].screen_rows * scrollback_pages * CHR, M_DEVBUF, - M_WAITOK)) == NULL) - { - printf("pcvt: scrollback memory malloc failed\n"); - } + Scrollbuffer = (u_short *)malloc(vs[0].maxcol * vs[0].screen_rows * + scrollback_pages * CHR, M_DEVBUF, + M_WAITOK); /* * Copy buffer must be 1 character wider than the screen because we @@ -1274,24 +1271,13 @@ vt_coldmalloc(void) */ Copybuffer_size = (vs[0].maxcol + 1) * vs[0].screen_rows; - if ((Copybuffer = (char *)malloc(Copybuffer_size, M_DEVBUF, M_WAITOK)) - == NULL) - { - printf("pcvt: copy memory malloc failed\n"); - Copybuffer_size = 0; - } + Copybuffer = (char *)malloc(Copybuffer_size, M_DEVBUF, M_WAITOK); for(nscr = 0; nscr < PCVT_NSCREENS; nscr++) { - if((vs[nscr].Memory = - (u_short *)malloc(screen_max_size * 2, M_DEVBUF, M_WAITOK)) - == NULL) - { - printf("pcvt: screen memory malloc failed, " - "NSCREEN=%d, nscr=%d\n", - PCVT_NSCREENS, nscr); - break; - } + vs[nscr].Memory = + (u_short *)malloc(screen_max_size * 2, M_DEVBUF, M_WAITOK); + vs[nscr].Scrollback = Scrollbuffer; if(nscr != 0) { diff --git a/sys/arch/i386/isa/pcvt/pcvt_sup.c b/sys/arch/i386/isa/pcvt/pcvt_sup.c index 5c94032a94a..5c96ae69c12 100644 --- a/sys/arch/i386/isa/pcvt/pcvt_sup.c +++ b/sys/arch/i386/isa/pcvt/pcvt_sup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pcvt_sup.c,v 1.17 2000/09/28 17:45:42 aaron Exp $ */ +/* $OpenBSD: pcvt_sup.c,v 1.18 2001/05/16 12:49:46 ho Exp $ */ /* * Copyright (c) 1992, 1995 Hellmuth Michaelis and Joerg Wunsch. @@ -1356,18 +1356,11 @@ loadchar(int fontset, int character, int char_scanlines, u_char *char_table) saved_charsets[fontset] = (u_char *)malloc(32 * 256, M_DEVBUF, M_WAITOK); - if((bak = saved_charsets[fontset])) - { - /* make a backup copy of this char */ - bak += (character * 32); - bzero(bak, 32); - bcopy(char_table, bak, char_scanlines); - } -#ifdef DIAGNOSTIC - else - panic("pcvt loadchar: no backup buffer"); -#endif /* DIAGNOSTIC */ - + bak = saved_charsets[fontset]; + /* make a backup copy of this char */ + bak += (character * 32); + bzero(bak, 32); + bcopy(char_table, bak, char_scanlines); #endif /* PCVT_BACKUP_FONTS */ } diff --git a/sys/arch/mac68k/dev/if_sn.c b/sys/arch/mac68k/dev/if_sn.c index 649512bde69..31498f76d61 100644 --- a/sys/arch/mac68k/dev/if_sn.c +++ b/sys/arch/mac68k/dev/if_sn.c @@ -132,11 +132,6 @@ snsetup(sc, lladdr) * to do that? */ sc->space = malloc((SN_NPAGES + 1) * NBPG, M_DEVBUF, M_WAITOK); - if (sc->space == NULL) { - printf ("%s: memory allocation for descriptors failed\n", - sc->sc_dev.dv_xname); - return (1); - } /* * Put the pup in reset mode (sninit() will fix it later), diff --git a/sys/arch/mvme68k/dev/flash.c b/sys/arch/mvme68k/dev/flash.c index 38817227130..3bafb882bea 100644 --- a/sys/arch/mvme68k/dev/flash.c +++ b/sys/arch/mvme68k/dev/flash.c @@ -1,4 +1,4 @@ -/* $OpenBSD: flash.c,v 1.7 2000/03/26 23:31:59 deraadt Exp $ */ +/* $OpenBSD: flash.c,v 1.8 2001/05/16 12:49:46 ho Exp $ */ /* * Copyright (c) 1995 Theo de Raadt @@ -207,8 +207,6 @@ flashsavezone(sc, start) u_char *zone; zone = (u_char *)malloc(sc->sc_zonesize, M_TEMP, M_WAITOK); - if (!zone) - return (NULL); sc->sc_vaddr[0] = FLCMD_RESET; bcopy((u_char *)&sc->sc_vaddr[start], zone, sc->sc_zonesize); return (zone); @@ -394,8 +392,6 @@ flashwrite(dev, uio, flags) int zonestart, zoneoff; cmpbuf = (u_char *)malloc(sc->sc_zonesize, M_TEMP, M_WAITOK); - if (!cmpbuf) - return (ENOMEM); while (uio->uio_resid > 0 && error == 0) { iov = uio->uio_iov; diff --git a/sys/arch/mvme68k/dev/vs.c b/sys/arch/mvme68k/dev/vs.c index 933ea6b5820..068010195ac 100644 --- a/sys/arch/mvme68k/dev/vs.c +++ b/sys/arch/mvme68k/dev/vs.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vs.c,v 1.2 2000/06/10 19:53:23 deraadt Exp $ */ +/* $OpenBSD: vs.c,v 1.3 2001/05/16 12:49:47 ho Exp $ */ /* * Copyright (c) 1999 Steve Murphree, Jr. @@ -839,10 +839,6 @@ vs_alloc_scatter_gather(void) M328_SG sg; MALLOC(sg, M328_SG, sizeof(struct m328_sg), M_DEVBUF, M_WAITOK); - assert ( sg ); - if ( !sg ) { - panic ("Memory for scatter_gather_list not available"); - } bzero(sg, sizeof(struct m328_sg)); return (sg); diff --git a/sys/arch/mvme88k/dev/vs.c b/sys/arch/mvme88k/dev/vs.c index 51587ba9aa7..da71e0ef0ca 100644 --- a/sys/arch/mvme88k/dev/vs.c +++ b/sys/arch/mvme88k/dev/vs.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vs.c,v 1.5 2001/03/09 05:44:39 smurph Exp $ */ +/* $OpenBSD: vs.c,v 1.6 2001/05/16 12:49:47 ho Exp $ */ /* * Copyright (c) 1999 Steve Murphree, Jr. @@ -927,10 +927,6 @@ vs_alloc_scatter_gather(void) M328_SG sg; MALLOC(sg, M328_SG, sizeof(struct m328_sg), M_DEVBUF, M_WAITOK); - assert ( sg ); - if ( !sg ) { - panic ("Memory for scatter_gather_list not available"); - } bzero(sg, sizeof(struct m328_sg)); return (sg); diff --git a/sys/arch/mvme88k/mvme88k/pmap.c b/sys/arch/mvme88k/mvme88k/pmap.c index c7798c630b7..6bf3e432933 100644 --- a/sys/arch/mvme88k/mvme88k/pmap.c +++ b/sys/arch/mvme88k/mvme88k/pmap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pmap.c,v 1.26 2001/05/09 15:31:25 art Exp $ */ +/* $OpenBSD: pmap.c,v 1.27 2001/05/16 12:49:47 ho Exp $ */ /* * Copyright (c) 1996 Nivas Madhur * All rights reserved. @@ -1674,9 +1674,6 @@ pmap_create(vm_size_t size) CHECK_PMAP_CONSISTENCY("pmap_create"); p = (pmap_t)malloc(sizeof(*p), M_VMPMAP, M_WAITOK); - if (p == PMAP_NULL) { - panic("pmap_create: cannot allocate a pmap"); - } bzero(p, sizeof(*p)); pmap_pinit(p); diff --git a/sys/arch/sparc/dev/isp_sbus.c b/sys/arch/sparc/dev/isp_sbus.c index 6f52ca6f6be..ccbe0d68712 100644 --- a/sys/arch/sparc/dev/isp_sbus.c +++ b/sys/arch/sparc/dev/isp_sbus.c @@ -1,4 +1,4 @@ -/* $OpenBSD: isp_sbus.c,v 1.16 2001/04/04 22:06:22 mjacob Exp $ */ +/* $OpenBSD: isp_sbus.c,v 1.17 2001/05/16 12:49:48 ho Exp $ */ /* * SBus specific probe and attach routines for Qlogic ISP SCSI adapters. * @@ -302,17 +302,9 @@ isp_sbus_mbxdma(struct ispsoftc *isp) len = isp->isp_maxcmds * sizeof (XS_T); isp->isp_xflist = (XS_T **) malloc(len, M_DEVBUF, M_WAITOK); - if (isp->isp_xflist == NULL) { - printf("%s: cannot malloc xflist array\n", isp->isp_name); - return (1); - } bzero(isp->isp_xflist, len); len = isp->isp_maxcmds * sizeof (vaddr_t); sbc->sbus_kdma_allocs = (vaddr_t *) malloc(len, M_DEVBUF, M_WAITOK); - if (sbc->sbus_kdma_allocs == NULL) { - printf("%s: cannot malloc sbus_kdma_allocs\n", isp->isp_name); - return (1); - } bzero(sbc->sbus_kdma_allocs, len); /* diff --git a/sys/arch/sparc/dev/magma.c b/sys/arch/sparc/dev/magma.c index dca22b6734c..5571f99a977 100644 --- a/sys/arch/sparc/dev/magma.c +++ b/sys/arch/sparc/dev/magma.c @@ -1,4 +1,4 @@ -/* $OpenBSD: magma.c,v 1.7 2001/03/24 10:07:19 ho Exp $ */ +/* $OpenBSD: magma.c,v 1.8 2001/05/16 12:49:48 ho Exp $ */ /* * magma.c * @@ -1591,8 +1591,6 @@ int gotdata = 0; buflen = min(uio->uio_resid, mp->mp_burst); buffer = malloc(buflen, M_DEVBUF, M_WAITOK); - if( buffer == NULL ) - return(ENOMEM); SET(mp->mp_flags, MBPPF_UIO); diff --git a/sys/arch/sparc/sparc/clock.c b/sys/arch/sparc/sparc/clock.c index dbf34309535..e84b775060e 100644 --- a/sys/arch/sparc/sparc/clock.c +++ b/sys/arch/sparc/sparc/clock.c @@ -1,4 +1,4 @@ -/* $OpenBSD: clock.c,v 1.13 1999/09/29 13:54:03 art Exp $ */ +/* $OpenBSD: clock.c,v 1.14 2001/05/16 12:49:48 ho Exp $ */ /* $NetBSD: clock.c,v 1.52 1997/05/24 20:16:05 pk Exp $ */ /* @@ -1261,10 +1261,6 @@ eeprom_uio(uio) * temporary buffer. */ buf = malloc(EEPROM_SIZE, M_DEVBUF, M_WAITOK); - if (buf == NULL) { - error = EAGAIN; - goto out; - } if (uio->uio_rw == UIO_READ) for (bcnt = 0; bcnt < EEPROM_SIZE; ++bcnt) diff --git a/sys/arch/sun3/dev/eeprom.c b/sys/arch/sun3/dev/eeprom.c index d4e36dff344..61672907da6 100644 --- a/sys/arch/sun3/dev/eeprom.c +++ b/sys/arch/sun3/dev/eeprom.c @@ -1,4 +1,4 @@ -/* $OpenBSD: eeprom.c,v 1.7 2001/01/04 22:34:42 miod Exp $ */ +/* $OpenBSD: eeprom.c,v 1.8 2001/05/16 12:49:48 ho Exp $ */ /* $NetBSD: eeprom.c,v 1.8 1996/03/26 15:16:06 gwr Exp $ */ /* @@ -175,12 +175,7 @@ eeprom_uio(uio) va = eeprom_va; if (uio->uio_rw != UIO_READ) { /* Write requires a temporary buffer. */ - buf = malloc(OBIO_EEPROM_SIZE, M_DEVBUF, M_WAITOK); - if (!buf) { - error = EAGAIN; - goto out; - } - va = buf; + va = buf = malloc(OBIO_EEPROM_SIZE, M_DEVBUF, M_WAITOK); } if ((error = uiomove(va + off, (int)cnt, uio)) != 0) diff --git a/sys/arch/sun3/dev/si.c b/sys/arch/sun3/dev/si.c index 5c573989874..917e3167cd2 100644 --- a/sys/arch/sun3/dev/si.c +++ b/sys/arch/sun3/dev/si.c @@ -1,4 +1,4 @@ -/* $OpenBSD: si.c,v 1.13 2000/09/21 21:25:16 miod Exp $ */ +/* $OpenBSD: si.c,v 1.14 2001/05/16 12:49:49 ho Exp $ */ /* $NetBSD: si.c,v 1.31 1996/11/20 18:56:59 gwr Exp $ */ /*- @@ -203,8 +203,6 @@ si_attach(sc) i = SCI_OPENINGS * sizeof(struct si_dma_handle); sc->sc_dma = (struct si_dma_handle *) malloc(i, M_DEVBUF, M_WAITOK); - if (sc->sc_dma == NULL) - panic("si: dvma_malloc failed"); for (i = 0; i < SCI_OPENINGS; i++) sc->sc_dma[i].dh_flags = 0; diff --git a/sys/compat/hpux/hpux_net.c b/sys/compat/hpux/hpux_net.c index 6703a495ac6..be984ddd558 100644 --- a/sys/compat/hpux/hpux_net.c +++ b/sys/compat/hpux/hpux_net.c @@ -1,4 +1,4 @@ -/* $OpenBSD: hpux_net.c,v 1.4 2000/11/10 18:15:44 art Exp $ */ +/* $OpenBSD: hpux_net.c,v 1.5 2001/05/16 12:50:20 ho Exp $ */ /* $NetBSD: hpux_net.c,v 1.14 1997/04/01 19:59:02 scottr Exp $ */ /* @@ -221,8 +221,6 @@ hpux_sys_setsockopt(p, v, retval) return (EINVAL); if (SCARG(uap, val)) { m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) - return (ENOBUFS); if ((error = copyin(SCARG(uap, val), mtod(m, caddr_t), (u_int)SCARG(uap, valsize)))) { (void) m_free(m); @@ -237,11 +235,9 @@ hpux_sys_setsockopt(p, v, retval) socksetsize(SCARG(uap, valsize), m); } else if (SCARG(uap, name) == ~SO_LINGER) { m = m_get(M_WAIT, MT_SOOPTS); - if (m) { - SCARG(uap, name) = SO_LINGER; - mtod(m, struct linger *)->l_onoff = 0; - m->m_len = sizeof(struct linger); - } + SCARG(uap, name) = SO_LINGER; + mtod(m, struct linger *)->l_onoff = 0; + m->m_len = sizeof(struct linger); } return (sosetopt((struct socket *)fp->f_data, SCARG(uap, level), SCARG(uap, name), m)); @@ -265,8 +261,6 @@ hpux_sys_setsockopt2(p, v, retval) return (EINVAL); if (SCARG(uap, val)) { m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) - return (ENOBUFS); if ((error = copyin(SCARG(uap, val), mtod(m, caddr_t), (u_int)SCARG(uap, valsize)))) { (void) m_free(m); diff --git a/sys/compat/linux/linux_getcwd.c b/sys/compat/linux/linux_getcwd.c index 6a66a2642c1..93a032df3e9 100644 --- a/sys/compat/linux/linux_getcwd.c +++ b/sys/compat/linux/linux_getcwd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: linux_getcwd.c,v 1.1 2000/03/27 22:38:12 jasoni Exp $ */ +/* $OpenBSD: linux_getcwd.c,v 1.2 2001/05/16 12:50:21 ho Exp $ */ /* $NetBSD: vfs_getcwd.c,v 1.3.2.3 1999/07/11 10:24:09 sommerfeld Exp $ */ /*- @@ -585,8 +585,6 @@ int linux_sys_getcwd(p, v, retval) return ERANGE; path = (char *)malloc(len, M_TEMP, M_WAITOK); - if (!path) - return ENOMEM; bp = &path[len]; bend = bp; diff --git a/sys/compat/netbsd/netbsd_getcwd.c b/sys/compat/netbsd/netbsd_getcwd.c index fe9bec0ecf3..629c24f1218 100644 --- a/sys/compat/netbsd/netbsd_getcwd.c +++ b/sys/compat/netbsd/netbsd_getcwd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: netbsd_getcwd.c,v 1.3 1999/09/26 11:15:43 kstailey Exp $ */ +/* $OpenBSD: netbsd_getcwd.c,v 1.4 2001/05/16 12:50:21 ho Exp $ */ /* $NetBSD: vfs_getcwd.c,v 1.3.2.3 1999/07/11 10:24:09 sommerfeld Exp $ */ /*- @@ -579,8 +579,6 @@ int netbsd_sys___getcwd(p, v, retval) return ERANGE; path = (char *)malloc(len, M_TEMP, M_WAITOK); - if (!path) - return ENOMEM; bp = &path[len]; bend = bp; diff --git a/sys/compat/sunos/sunos_misc.c b/sys/compat/sunos/sunos_misc.c index dbef9174ff2..b05b8204979 100644 --- a/sys/compat/sunos/sunos_misc.c +++ b/sys/compat/sunos/sunos_misc.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sunos_misc.c,v 1.26 2001/05/05 21:26:42 art Exp $ */ +/* $OpenBSD: sunos_misc.c,v 1.27 2001/05/16 12:50:20 ho Exp $ */ /* $NetBSD: sunos_misc.c,v 1.65 1996/04/22 01:44:31 christos Exp $ */ /* @@ -545,8 +545,6 @@ sunos_sys_setsockopt(p, v, retval) #define SO_DONTLINGER (~SO_LINGER) if (SCARG(uap, name) == SO_DONTLINGER) { m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) - return (ENOBUFS); mtod(m, struct linger *)->l_onoff = 0; m->m_len = sizeof(struct linger); return (sosetopt((struct socket *)fp->f_data, SCARG(uap, level), @@ -575,8 +573,6 @@ sunos_sys_setsockopt(p, v, retval) return (EINVAL); if (SCARG(uap, val)) { m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) - return (ENOBUFS); error = copyin(SCARG(uap, val), mtod(m, caddr_t), (u_int)SCARG(uap, valsize)); if (error) { diff --git a/sys/compat/ultrix/ultrix_misc.c b/sys/compat/ultrix/ultrix_misc.c index 5afb61b3363..7e8f7c560f7 100644 --- a/sys/compat/ultrix/ultrix_misc.c +++ b/sys/compat/ultrix/ultrix_misc.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ultrix_misc.c,v 1.17 2001/05/05 21:26:44 art Exp $ */ +/* $OpenBSD: ultrix_misc.c,v 1.18 2001/05/16 12:50:21 ho Exp $ */ /* $NetBSD: ultrix_misc.c,v 1.23 1996/04/07 17:23:04 jonathan Exp $ */ /* @@ -372,8 +372,6 @@ ultrix_sys_setsockopt(p, v, retval) #define SO_DONTLINGER (~SO_LINGER) if (SCARG(uap, name) == SO_DONTLINGER) { m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) - return (ENOBUFS); mtod(m, struct linger *)->l_onoff = 0; m->m_len = sizeof(struct linger); return (sosetopt((struct socket *)fp->f_data, SCARG(uap, level), @@ -383,8 +381,6 @@ ultrix_sys_setsockopt(p, v, retval) return (EINVAL); if (SCARG(uap, val)) { m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) - return (ENOBUFS); if ((error = copyin(SCARG(uap, val), mtod(m, caddr_t), (u_int)SCARG(uap, valsize))) != 0) { (void) m_free(m); diff --git a/sys/crypto/cryptodev.c b/sys/crypto/cryptodev.c index 1f23f232453..6c4fc16b2a7 100644 --- a/sys/crypto/cryptodev.c +++ b/sys/crypto/cryptodev.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cryptodev.c,v 1.7 2001/05/15 09:00:23 deraadt Exp $ */ +/* $OpenBSD: cryptodev.c,v 1.8 2001/05/16 12:54:34 ho Exp $ */ /* * Copyright (c) 2001 Theo de Raadt @@ -278,8 +278,6 @@ crypto_op(struct csession *cse, struct crypt_op *cop, struct proc *p) bzero(&cse->iovec, sizeof(cse->iovec)); cse->uio.uio_iov[0].iov_len = cop->len; cse->uio.uio_iov[0].iov_base = malloc(cop->len, M_XDATA, M_WAITOK); - if (cse->uio.uio_iov[0].iov_base == NULL) - return (ENOMEM); for (i = 0; i < cse->uio.uio_iovcnt; i++) cse->uio.uio_resid += cse->uio.uio_iov[0].iov_len; diff --git a/sys/dev/ic/ac97.c b/sys/dev/ic/ac97.c index 3534f0b0832..57e0a4dd01c 100644 --- a/sys/dev/ic/ac97.c +++ b/sys/dev/ic/ac97.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ac97.c,v 1.16 2001/05/16 08:19:21 deraadt Exp $ */ +/* $OpenBSD: ac97.c,v 1.17 2001/05/16 12:51:48 ho Exp $ */ /* * Copyright (c) 1999, 2000 Constantine Sapuntzakis @@ -583,9 +583,6 @@ ac97_attach(host_if) int error, i; as = malloc(sizeof(struct ac97_softc), M_DEVBUF, M_WAITOK); - if (!as) - return (ENOMEM); - bzero(as, sizeof(*as)); as->codec_if.vtbl = &ac97civ; diff --git a/sys/dev/ic/adw.c b/sys/dev/ic/adw.c index 1d79611dd7d..da0dd14582d 100644 --- a/sys/dev/ic/adw.c +++ b/sys/dev/ic/adw.c @@ -1,4 +1,4 @@ -/* $OpenBSD: adw.c,v 1.17 2001/04/11 04:05:15 krw Exp $ */ +/* $OpenBSD: adw.c,v 1.18 2001/05/16 12:51:48 ho Exp $ */ /* $NetBSD: adw.c,v 1.23 2000/05/27 18:24:50 dante Exp $ */ /* @@ -219,13 +219,9 @@ adw_alloc_carriers(sc) /* * Allocate the control structure. */ - sc->sc_control->carriers = malloc(sizeof(ADW_CARRIER) * ADW_MAX_CARRIER, - M_DEVBUF, M_WAITOK); - if(!sc->sc_control->carriers) { - printf("%s: malloc() failed in allocating carrier structures\n", - sc->sc_dev.dv_xname); - return (ENOMEM); - } + sc->sc_control->carriers = + malloc(sizeof(ADW_CARRIER) * ADW_MAX_CARRIER, M_DEVBUF, + M_WAITOK); if ((error = bus_dmamem_alloc(sc->sc_dmat, sizeof(ADW_CARRIER) * ADW_MAX_CARRIER, diff --git a/sys/dev/pci/auvia.c b/sys/dev/pci/auvia.c index 3a2dbfcdaed..c534a1a9933 100644 --- a/sys/dev/pci/auvia.c +++ b/sys/dev/pci/auvia.c @@ -1,4 +1,4 @@ -/* $OpenBSD: auvia.c,v 1.7 2001/05/16 05:22:58 mickey Exp $ */ +/* $OpenBSD: auvia.c,v 1.8 2001/05/16 12:51:49 ho Exp $ */ /* $NetBSD: auvia.c,v 1.7 2000/11/15 21:06:33 jdolecek Exp $ */ /*- @@ -820,11 +820,6 @@ auvia_build_dma_ops(struct auvia_softc *sc, struct auvia_softc_chan *ch, ch->sc_dma_ops = auvia_malloc(sc, 0, sizeof(struct auvia_dma_op) * segs, M_DEVBUF, M_WAITOK); - if (ch->sc_dma_ops == NULL) { - printf("%s: couldn't build dmaops\n", sc->sc_dev.dv_xname); - return 1; - } - for (dp = sc->sc_dmas; dp && dp->addr != (void *)(ch->sc_dma_ops); dp = dp->next) diff --git a/sys/dev/pci/isp_pci.c b/sys/dev/pci/isp_pci.c index 47e940d00bf..6612d54a7df 100644 --- a/sys/dev/pci/isp_pci.c +++ b/sys/dev/pci/isp_pci.c @@ -1,4 +1,4 @@ -/* $OpenBSD: isp_pci.c,v 1.18 2001/04/04 22:05:37 mjacob Exp $ */ +/* $OpenBSD: isp_pci.c,v 1.19 2001/05/16 12:51:49 ho Exp $ */ /* * PCI specific probe and attach routines for Qlogic ISP SCSI adapters. * @@ -769,17 +769,9 @@ isp_pci_mbxdma(struct ispsoftc *isp) len = isp->isp_maxcmds * sizeof (XS_T); isp->isp_xflist = (XS_T **) malloc(len, M_DEVBUF, M_WAITOK); - if (isp->isp_xflist == NULL) { - printf("%s: cannot malloc xflist array\n", isp->isp_name); - return (1); - } bzero(isp->isp_xflist, len); len = isp->isp_maxcmds * sizeof (bus_dmamap_t); pci->pci_xfer_dmap = (bus_dmamap_t *) malloc(len, M_DEVBUF, M_WAITOK); - if (pci->pci_xfer_dmap == NULL) { - printf("%s: cannot dma xfer map array\n", isp->isp_name); - return (1); - } /* * Allocate and map the request queue. diff --git a/sys/dev/pci/maestro.c b/sys/dev/pci/maestro.c index 78bd8776e14..58d4a6ea3cc 100644 --- a/sys/dev/pci/maestro.c +++ b/sys/dev/pci/maestro.c @@ -1,4 +1,4 @@ -/* $OpenBSD: maestro.c,v 1.4 2001/03/13 01:45:56 deraadt Exp $ */ +/* $OpenBSD: maestro.c,v 1.5 2001/05/16 12:51:50 ho Exp $ */ /* $FreeBSD: /c/ncvs/src/sys/dev/sound/pci/maestro.c,v 1.3 2000/11/21 12:22:11 julian Exp $ */ /* * FreeBSD's ESS Agogo/Maestro driver @@ -1508,8 +1508,6 @@ salloc_new(addr, size, nzones) MALLOC(pool, salloc_t, sizeof *pool + nzones * sizeof pool->zones[0], M_TEMP, M_WAITOK); - if (pool == NULL) - return NULL; SLIST_INIT(&pool->free); SLIST_INIT(&pool->used); SLIST_INIT(&pool->spare); diff --git a/sys/dev/pci/ncr.c b/sys/dev/pci/ncr.c index e2bb618bdce..2e6b1651c89 100644 --- a/sys/dev/pci/ncr.c +++ b/sys/dev/pci/ncr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ncr.c,v 1.54 2001/04/06 04:42:07 csapuntz Exp $ */ +/* $OpenBSD: ncr.c,v 1.55 2001/05/16 12:51:49 ho Exp $ */ /* $NetBSD: ncr.c,v 1.63 1997/09/23 02:39:15 perry Exp $ */ /************************************************************************** @@ -1464,7 +1464,7 @@ static void ncr_attach (pcici_t tag, int unit); #if 0 static char ident[] = - "\n$OpenBSD: ncr.c,v 1.54 2001/04/06 04:42:07 csapuntz Exp $\n"; + "\n$OpenBSD: ncr.c,v 1.55 2001/05/16 12:51:49 ho Exp $\n"; #endif static const u_long ncr_version = NCR_VERSION * 11 @@ -3662,7 +3662,6 @@ ncr_attach(parent, self, aux) np->sc_pc = pc; np->ccb = (ccb_p) malloc (sizeof (struct ccb), M_DEVBUF, M_WAITOK); - if (!np->ccb) return; #if defined(__mips__) pci_sync_cache(pc, (vm_offset_t)np->ccb, sizeof (struct ccb)); np->ccb = (struct ccb *)PHYS_TO_UNCACHED(NCR_KVATOPHYS(np, np->ccb)); @@ -3768,13 +3767,11 @@ static void ncr_attach (pcici_t config_id, int unit) if (!np) { np = (ncb_p) malloc (sizeof (struct ncb), M_DEVBUF, M_WAITOK); - if (!np) return; ncrp[unit]=np; } bzero (np, sizeof (*np)); np->ccb = (ccb_p) malloc (sizeof (struct ccb), M_DEVBUF, M_WAITOK); - if (!np->ccb) return; #if defined(__mips__) pci_sync_cache(pc, (vm_offset_t)np->ccb, sizeof (struct ccb)); np->ccb = (struct ccb *)PHYS_TO_UNCACHED(NCR_KVATOPHYS(np, np->ccb)); diff --git a/sys/dev/pci/pccbb.c b/sys/dev/pci/pccbb.c index 40558d3263e..b661b6094cf 100644 --- a/sys/dev/pci/pccbb.c +++ b/sys/dev/pci/pccbb.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pccbb.c,v 1.18 2001/05/01 02:19:46 mickey Exp $ */ +/* $OpenBSD: pccbb.c,v 1.19 2001/05/16 12:51:49 ho Exp $ */ /* $NetBSD: pccbb.c,v 1.42 2000/06/16 23:41:35 cgd Exp $ */ /* @@ -1688,11 +1688,8 @@ pccbb_intr_establish(sc, irq, level, func, arg) /* * Allocate a room for interrupt handler structure. */ - if (NULL == (newpil = - (struct pccbb_intrhand_list *)malloc(sizeof(struct - pccbb_intrhand_list), M_DEVBUF, M_WAITOK))) { - return NULL; - } + newpil = (struct pccbb_intrhand_list *) + malloc(sizeof(struct pccbb_intrhand_list), M_DEVBUF, M_WAITOK); newpil->pil_func = func; newpil->pil_arg = arg; diff --git a/sys/dev/ramdisk.c b/sys/dev/ramdisk.c index aaf28e73aa1..43ee642e441 100644 --- a/sys/dev/ramdisk.c +++ b/sys/dev/ramdisk.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ramdisk.c,v 1.11 1999/11/16 09:24:59 art Exp $ */ +/* $OpenBSD: ramdisk.c,v 1.12 2001/05/16 12:51:48 ho Exp $ */ /* $NetBSD: ramdisk.c,v 1.8 1996/04/12 08:30:09 leo Exp $ */ /* @@ -161,10 +161,6 @@ rdattach(n) for (i = 0; i < n; i++) { sc = malloc(sizeof(*sc), M_DEVBUF, M_WAITOK); - if (!sc) { - printf("ramdisk: malloc for attach failed!\n"); - return; - } bzero((caddr_t)sc, sizeof(*sc)); ramdisk_devs[i] = sc; sc->sc_dev.dv_unit = i; diff --git a/sys/dev/wscons/wsdisplay.c b/sys/dev/wscons/wsdisplay.c index d931bb8580e..dbfa6441287 100644 --- a/sys/dev/wscons/wsdisplay.c +++ b/sys/dev/wscons/wsdisplay.c @@ -1,4 +1,4 @@ -/* $OpenBSD: wsdisplay.c,v 1.26 2001/05/08 22:28:43 mickey Exp $ */ +/* $OpenBSD: wsdisplay.c,v 1.27 2001/05/16 12:51:50 ho Exp $ */ /* $NetBSD: wsdisplay.c,v 1.37.4.1 2000/06/30 16:27:53 simonb Exp $ */ /* @@ -266,8 +266,6 @@ wsscreen_attach(sc, console, emul, type, cookie, ccol, crow, defattr) struct wsscreen *scr; scr = malloc(sizeof(struct wsscreen), M_DEVBUF, M_WAITOK); - if (!scr) - return (NULL); if (console) { dconf = &wsdisplay_console_conf; diff --git a/sys/dev/wscons/wsdisplay_compat_usl.c b/sys/dev/wscons/wsdisplay_compat_usl.c index a45872a21d5..57510d92ede 100644 --- a/sys/dev/wscons/wsdisplay_compat_usl.c +++ b/sys/dev/wscons/wsdisplay_compat_usl.c @@ -1,4 +1,4 @@ -/* $OpenBSD: wsdisplay_compat_usl.c,v 1.6 2001/05/16 05:07:51 millert Exp $ */ +/* $OpenBSD: wsdisplay_compat_usl.c,v 1.7 2001/05/16 12:51:50 ho Exp $ */ /* $NetBSD: wsdisplay_compat_usl.c,v 1.12 2000/03/23 07:01:47 thorpej Exp $ */ /* @@ -101,8 +101,6 @@ usl_sync_init(scr, sdp, p, acqsig, relsig, frsig) int res; sd = malloc(sizeof(struct usl_syncdata), M_DEVBUF, M_WAITOK); - if (!sd) - return (ENOMEM); sd->s_scr = scr; sd->s_proc = p; sd->s_pid = p->p_pid; diff --git a/sys/kern/kern_lkm.c b/sys/kern/kern_lkm.c index dfe83208002..27f71b21d9b 100644 --- a/sys/kern/kern_lkm.c +++ b/sys/kern/kern_lkm.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_lkm.c,v 1.29 2001/05/05 20:57:00 art Exp $ */ +/* $OpenBSD: kern_lkm.c,v 1.30 2001/05/16 12:52:58 ho Exp $ */ /* $NetBSD: kern_lkm.c,v 1.31 1996/03/31 21:40:27 christos Exp $ */ /* @@ -155,13 +155,10 @@ lkmalloc() struct lkm_table *ret = NULL; MALLOC(ret, struct lkm_table *, sizeof(*ret), M_DEVBUF, M_WAITOK); - if (ret != NULL) { - ret->refcnt = - ret->depcnt = 0; - ret->id = nlkms++; - ret->sym_id = -1; - TAILQ_INSERT_TAIL(&lkmods, ret, list); - } + ret->refcnt = ret->depcnt = 0; + ret->id = nlkms++; + ret->sym_id = -1; + TAILQ_INSERT_TAIL(&lkmods, ret, list); return ret; } diff --git a/sys/kern/sys_generic.c b/sys/kern/sys_generic.c index 595071f1258..5c2b5a3e2c5 100644 --- a/sys/kern/sys_generic.c +++ b/sys/kern/sys_generic.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sys_generic.c,v 1.28 2000/11/10 18:15:47 art Exp $ */ +/* $OpenBSD: sys_generic.c,v 1.29 2001/05/16 12:52:58 ho Exp $ */ /* $NetBSD: sys_generic.c,v 1.24 1996/03/29 00:25:32 cgd Exp $ */ /* @@ -691,10 +691,7 @@ sys_select(p, v, retval) if (SCARG(uap, nd) > FD_SETSIZE) { caddr_t mbits; - if ((mbits = malloc(ni * 6, M_TEMP, M_WAITOK)) == NULL) { - error = EINVAL; - goto cleanup; - } + mbits = malloc(ni * 6, M_TEMP, M_WAITOK); bzero(mbits, ni * 6); pibits[0] = (fd_set *)&mbits[ni * 0]; pibits[1] = (fd_set *)&mbits[ni * 1]; @@ -779,7 +776,6 @@ done: #undef putbits } -cleanup: if (pibits[0] != &bits[0]) free(pibits[0], M_TEMP); return (error); diff --git a/sys/kern/tty_subr.c b/sys/kern/tty_subr.c index b3d0ceae796..dbd1a48e201 100644 --- a/sys/kern/tty_subr.c +++ b/sys/kern/tty_subr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tty_subr.c,v 1.10 2001/01/25 03:50:53 todd Exp $ */ +/* $OpenBSD: tty_subr.c,v 1.11 2001/05/16 12:52:58 ho Exp $ */ /* $NetBSD: tty_subr.c,v 1.13 1996/02/09 19:00:43 christos Exp $ */ /* @@ -94,17 +94,10 @@ clalloc(clp, size, quot) { clp->c_cs = malloc(size, M_TTYS, M_WAITOK); - if (!clp->c_cs) - return (-1); bzero(clp->c_cs, size); if (quot) { clp->c_cq = malloc(QMEM(size), M_TTYS, M_WAITOK); - if (!clp->c_cq) { - free(clp->c_cs, M_TTYS); - clp->c_cs = NULL; - return (-1); - } bzero(clp->c_cq, QMEM(size)); } else clp->c_cq = (u_char *)0; diff --git a/sys/kern/uipc_syscalls.c b/sys/kern/uipc_syscalls.c index fddbf376b58..9cba7d90786 100644 --- a/sys/kern/uipc_syscalls.c +++ b/sys/kern/uipc_syscalls.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uipc_syscalls.c,v 1.37 2001/05/14 13:43:54 art Exp $ */ +/* $OpenBSD: uipc_syscalls.c,v 1.38 2001/05/16 12:52:58 ho Exp $ */ /* $NetBSD: uipc_syscalls.c,v 1.19 1996/02/09 19:00:48 christos Exp $ */ /* @@ -479,15 +479,10 @@ sendit(p, s, mp, flags, retsize) register struct cmsghdr *cm; M_PREPEND(control, sizeof(*cm), M_WAIT); - if (control == 0) { - error = ENOBUFS; - goto bad; - } else { - cm = mtod(control, struct cmsghdr *); - cm->cmsg_len = control->m_len; - cm->cmsg_level = SOL_SOCKET; - cm->cmsg_type = SCM_RIGHTS; - } + cm = mtod(control, struct cmsghdr *); + cm->cmsg_len = control->m_len; + cm->cmsg_level = SOL_SOCKET; + cm->cmsg_type = SCM_RIGHTS; } #endif } else @@ -919,8 +914,6 @@ sys_getsockname(p, v, retval) return (error); so = (struct socket *)fp->f_data; m = m_getclr(M_WAIT, MT_SONAME); - if (m == NULL) - return (ENOBUFS); error = (*so->so_proto->pr_usrreq)(so, PRU_SOCKADDR, 0, m, 0); if (error) goto bad; @@ -965,8 +958,6 @@ sys_getpeername(p, v, retval) if (error) return (error); m = m_getclr(M_WAIT, MT_SONAME); - if (m == NULL) - return (ENOBUFS); error = (*so->so_proto->pr_usrreq)(so, PRU_PEERADDR, 0, m, 0); if (error) goto bad; diff --git a/sys/miscfs/procfs/procfs_status.c b/sys/miscfs/procfs/procfs_status.c index 5865eb0956f..e1d647efea2 100644 --- a/sys/miscfs/procfs/procfs_status.c +++ b/sys/miscfs/procfs/procfs_status.c @@ -1,4 +1,4 @@ -/* $OpenBSD: procfs_status.c,v 1.4 2000/12/18 18:44:28 provos Exp $ */ +/* $OpenBSD: procfs_status.c,v 1.5 2001/05/16 12:48:32 ho Exp $ */ /* $NetBSD: procfs_status.c,v 1.11 1996/03/16 23:52:50 christos Exp $ */ /* @@ -168,8 +168,6 @@ procfs_dostatus(curp, p, pfs, uio) len = procfs_stat_gen(p, NULL, 0); ps = malloc(len, M_TEMP, M_WAITOK); - if (!ps) - return (ENOMEM); (void) procfs_stat_gen(p, ps, len); len -= uio->uio_offset; diff --git a/sys/net/bpf.c b/sys/net/bpf.c index 5837e9618ef..f215c49dd60 100644 --- a/sys/net/bpf.c +++ b/sys/net/bpf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: bpf.c,v 1.25 2001/04/04 02:39:17 jason Exp $ */ +/* $OpenBSD: bpf.c,v 1.26 2001/05/16 12:53:34 ho Exp $ */ /* $NetBSD: bpf.c,v 1.33 1997/02/21 23:59:35 thorpej Exp $ */ /* @@ -169,8 +169,6 @@ bpf_movein(uio, linktype, mp, sockp) return (EIO); MGETHDR(m, M_WAIT, MT_DATA); - if (m == 0) - return (ENOBUFS); m->m_pkthdr.rcvif = 0; m->m_pkthdr.len = len - hlen; @@ -1145,14 +1143,7 @@ bpf_allocbufs(d) register struct bpf_d *d; { d->bd_fbuf = (caddr_t)malloc(d->bd_bufsize, M_DEVBUF, M_WAITOK); - if (d->bd_fbuf == 0) - return (ENOBUFS); - d->bd_sbuf = (caddr_t)malloc(d->bd_bufsize, M_DEVBUF, M_WAITOK); - if (d->bd_sbuf == 0) { - free(d->bd_fbuf, M_DEVBUF); - return (ENOBUFS); - } d->bd_slen = 0; d->bd_hlen = 0; return (0); diff --git a/sys/net/if_ppp.c b/sys/net/if_ppp.c index 46541d79576..d9214c62dcf 100644 --- a/sys/net/if_ppp.c +++ b/sys/net/if_ppp.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_ppp.c,v 1.16 2000/03/21 23:31:27 mickey Exp $ */ +/* $OpenBSD: if_ppp.c,v 1.17 2001/05/16 12:53:34 ho Exp $ */ /* $NetBSD: if_ppp.c,v 1.39 1997/05/17 21:11:59 christos Exp $ */ /* @@ -498,9 +498,6 @@ pppioctl(sc, cmd, data, flag, p) newcodelen = nbp->bf_len * sizeof(struct bpf_insn); if (newcodelen != 0) { MALLOC(newcode, struct bpf_insn *, newcodelen, M_DEVBUF, M_WAITOK); - if (newcode == 0) { - return EINVAL; /* or sumpin */ - } if ((error = copyin((caddr_t)nbp->bf_insns, (caddr_t)newcode, newcodelen)) != 0) { FREE(newcode, M_DEVBUF); diff --git a/sys/net/if_tun.c b/sys/net/if_tun.c index 87282337516..724cae4b2a1 100644 --- a/sys/net/if_tun.c +++ b/sys/net/if_tun.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_tun.c,v 1.33 2001/04/23 13:55:27 art Exp $ */ +/* $OpenBSD: if_tun.c,v 1.34 2001/05/16 12:53:34 ho Exp $ */ /* $NetBSD: if_tun.c,v 1.24 1996/05/07 02:40:48 thorpej Exp $ */ /* @@ -122,8 +122,6 @@ tunattach(n) ntun = n; tunctl = malloc(ntun * sizeof(*tunctl), M_DEVBUF, M_WAITOK); - if (!tunctl) - return; bzero(tunctl, ntun * sizeof(*tunctl)); for (i = 0; i < ntun; i++) { tunctl[i].tun_flags = TUN_INITED; diff --git a/sys/net/ppp_tty.c b/sys/net/ppp_tty.c index 9be485c26fb..3b41b062d5c 100644 --- a/sys/net/ppp_tty.c +++ b/sys/net/ppp_tty.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ppp_tty.c,v 1.10 2001/03/09 14:56:44 aaron Exp $ */ +/* $OpenBSD: ppp_tty.c,v 1.11 2001/05/16 12:53:34 ho Exp $ */ /* $NetBSD: ppp_tty.c,v 1.12 1997/03/24 21:23:10 christos Exp $ */ /* @@ -352,10 +352,7 @@ pppwrite(tp, uio, flag) return (EMSGSIZE); for (mp = &m0; uio->uio_resid; mp = &m->m_next) { MGET(m, M_WAIT, MT_DATA); - if ((*mp = m) == NULL) { - m_freem(m0); - return (ENOBUFS); - } + *mp = m; m->m_len = 0; if (uio->uio_resid >= MCLBYTES / 2) MCLGET(m, M_DONTWAIT); diff --git a/sys/net/rtsock.c b/sys/net/rtsock.c index 84cf734dd49..81901fb7c92 100644 --- a/sys/net/rtsock.c +++ b/sys/net/rtsock.c @@ -1,4 +1,4 @@ -/* $OpenBSD: rtsock.c,v 1.13 2001/01/19 06:37:37 itojun Exp $ */ +/* $OpenBSD: rtsock.c,v 1.14 2001/05/16 12:53:34 ho Exp $ */ /* $NetBSD: rtsock.c,v 1.18 1996/03/29 00:32:10 cgd Exp $ */ /* @@ -120,9 +120,8 @@ route_usrreq(so, req, m, nam, control) if (req == PRU_ATTACH) { MALLOC(rp, struct rawcb *, sizeof(*rp), M_PCB, M_WAITOK); - if ((so->so_pcb = rp) != NULL) - bzero(so->so_pcb, sizeof(*rp)); - + so->so_pcb = rp; + bzero(so->so_pcb, sizeof(*rp)); } if (req == PRU_DETACH && rp) { int af = rp->rcb_proto.sp_protocol; diff --git a/sys/netatalk/ddp_output.c b/sys/netatalk/ddp_output.c index 524b0a946b5..db460d35379 100644 --- a/sys/netatalk/ddp_output.c +++ b/sys/netatalk/ddp_output.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ddp_output.c,v 1.2 2000/06/16 21:47:15 provos Exp $ */ +/* $OpenBSD: ddp_output.c,v 1.3 2001/05/16 12:53:35 ho Exp $ */ /* * Copyright (c) 1990,1991 Regents of The University of Michigan. @@ -212,10 +212,6 @@ ddp_route( m, ro ) } } else { MGET( m0, M_WAIT, MT_HEADER ); - if ( m0 == 0 ) { - m_freem( m ); - return( ENOBUFS ); - } m0->m_next = m; m0->m_len = SZ_ELAPHDR; m = m0; diff --git a/sys/netccitt/pk_input.c b/sys/netccitt/pk_input.c index 7ed158197d9..3168d722bf8 100644 --- a/sys/netccitt/pk_input.c +++ b/sys/netccitt/pk_input.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pk_input.c,v 1.2 1996/03/04 07:36:41 niklas Exp $ */ +/* $OpenBSD: pk_input.c,v 1.3 2001/05/16 12:53:35 ho Exp $ */ /* $NetBSD: pk_input.c,v 1.7 1996/02/13 22:05:21 christos Exp $ */ /* @@ -120,8 +120,6 @@ pk_newlink(ia, llnext) */ size = sizeof(struct pkcb); pkp = (struct pkcb *) malloc(size, M_PCB, M_WAITOK); - if (pkp == 0) - return ((struct pkcb *) 0); bzero((caddr_t) pkp, size); pkp->pk_lloutput = pp->pr_output; pkp->pk_llctlinput = pp->pr_ctlinput; @@ -227,22 +225,16 @@ pk_resize(pkp) pkp->pk_maxlcn = xcp->xc_maxlcn; size = (pkp->pk_maxlcn + 1) * sizeof(struct pklcd *); pkp->pk_chan = malloc(size, M_IFADDR, M_WAITOK); - if (pkp->pk_chan) { - bzero((caddr_t) pkp->pk_chan, size); - /* - * Allocate a logical channel descriptor for lcn 0 - */ - if (dev_lcp == 0 && - (dev_lcp = pk_attach((struct socket *) 0)) == 0) - return (ENOBUFS); - dev_lcp->lcd_state = READY; - dev_lcp->lcd_pkp = pkp; - pkp->pk_chan[0] = dev_lcp; - } else { - if (dev_lcp) - pk_close(dev_lcp); + bzero((caddr_t) pkp->pk_chan, size); + /* + * Allocate a logical channel descriptor for lcn 0 + */ + if (dev_lcp == 0 && + (dev_lcp = pk_attach((struct socket *) 0)) == 0) return (ENOBUFS); - } + dev_lcp->lcd_state = READY; + dev_lcp->lcd_pkp = pkp; + pkp->pk_chan[0] = dev_lcp; } return 0; } diff --git a/sys/netccitt/pk_llcsubr.c b/sys/netccitt/pk_llcsubr.c index 2becb73d7fc..61b32fc48ed 100644 --- a/sys/netccitt/pk_llcsubr.c +++ b/sys/netccitt/pk_llcsubr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pk_llcsubr.c,v 1.4 2001/01/19 06:37:37 itojun Exp $ */ +/* $OpenBSD: pk_llcsubr.c,v 1.5 2001/05/16 12:53:35 ho Exp $ */ /* $NetBSD: pk_llcsubr.c,v 1.4 1996/02/13 22:05:26 christos Exp $ */ /* @@ -331,10 +331,8 @@ npaidb_enter(key, value, rt, link) npdl_netmask.sdl_data[i] = -1; nprt->rt_llinfo = malloc(size, M_PCB, M_WAITOK); - if (nprt->rt_llinfo) { - bzero(nprt->rt_llinfo, size); - ((struct npaidbentry *) (nprt->rt_llinfo))->np_rt = rt; - } + bzero(nprt->rt_llinfo, size); + ((struct npaidbentry *) (nprt->rt_llinfo))->np_rt = rt; } else nprt->rt_refcnt--; return nprt; diff --git a/sys/netccitt/pk_usrreq.c b/sys/netccitt/pk_usrreq.c index 8fa4c9b476c..24e1e334a74 100644 --- a/sys/netccitt/pk_usrreq.c +++ b/sys/netccitt/pk_usrreq.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pk_usrreq.c,v 1.2 1996/03/04 07:36:47 niklas Exp $ */ +/* $OpenBSD: pk_usrreq.c,v 1.3 2001/05/16 12:53:35 ho Exp $ */ /* $NetBSD: pk_usrreq.c,v 1.10 1996/02/13 22:05:43 christos Exp $ */ /* @@ -360,8 +360,6 @@ pk_control(so, cmd, data, ifp) if (ifa == (struct ifaddr *) 0) { MALLOC(ia, struct x25_ifaddr *, sizeof(*ia), M_IFADDR, M_WAITOK); - if (ia == 0) - return (ENOBUFS); bzero((caddr_t) ia, sizeof(*ia)); TAILQ_INSERT_TAIL(&ifp->if_addrlist, &ia->ia_ifa, ifa_list); @@ -570,7 +568,7 @@ pk_send(m, v) if (m->m_pkthdr.len > 32) error = EMSGSIZE; M_PREPEND(m, PKHEADERLN, M_WAITOK); - if (m == 0 || error) + if (error) goto bad; *(mtod(m, octet *)) = 0; xp = mtod(m, struct x25_packet *); diff --git a/sys/netinet/in.c b/sys/netinet/in.c index 3237049f8c2..17ef1ebcb7d 100644 --- a/sys/netinet/in.c +++ b/sys/netinet/in.c @@ -1,4 +1,4 @@ -/* $OpenBSD: in.c,v 1.18 2000/10/06 05:52:00 itojun Exp $ */ +/* $OpenBSD: in.c,v 1.19 2001/05/16 12:53:35 ho Exp $ */ /* $NetBSD: in.c,v 1.26 1996/02/13 23:41:39 christos Exp $ */ /* @@ -237,8 +237,6 @@ in_control(so, cmd, data, ifp) if (ia == (struct in_ifaddr *)0) { ia = (struct in_ifaddr *) malloc(sizeof *ia, M_IFADDR, M_WAITOK); - if (ia == (struct in_ifaddr *)0) - return (ENOBUFS); bzero((caddr_t)ia, sizeof *ia); TAILQ_INSERT_TAIL(&in_ifaddr, ia, ia_list); TAILQ_INSERT_TAIL(&ifp->if_addrlist, (struct ifaddr *)ia, diff --git a/sys/netinet/ip_output.c b/sys/netinet/ip_output.c index bef3b9beb6a..48d96b5452b 100644 --- a/sys/netinet/ip_output.c +++ b/sys/netinet/ip_output.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ip_output.c,v 1.89 2001/04/14 00:30:59 angelos Exp $ */ +/* $OpenBSD: ip_output.c,v 1.90 2001/05/16 12:53:36 ho Exp $ */ /* $NetBSD: ip_output.c,v 1.28 1996/02/13 23:43:07 christos Exp $ */ /* @@ -1224,8 +1224,6 @@ ip_setmoptions(optname, imop, m) imo = (struct ip_moptions *)malloc(sizeof(*imo), M_IPMOPTS, M_WAITOK); - if (imo == NULL) - return (ENOBUFS); *imop = imo; imo->imo_multicast_ifp = NULL; imo->imo_multicast_ttl = IP_DEFAULT_MULTICAST_TTL; diff --git a/sys/netinet6/in6.c b/sys/netinet6/in6.c index dc69615df10..86f324d2150 100644 --- a/sys/netinet6/in6.c +++ b/sys/netinet6/in6.c @@ -1,4 +1,4 @@ -/* $OpenBSD: in6.c,v 1.26 2001/02/16 15:58:50 itojun Exp $ */ +/* $OpenBSD: in6.c,v 1.27 2001/05/16 12:53:36 ho Exp $ */ /* $KAME: in6.c,v 1.176 2001/02/16 12:49:45 itojun Exp $ */ /* @@ -429,8 +429,6 @@ in6_control(so, cmd, data, ifp, p) if (ia == NULL) { ia = (struct in6_ifaddr *) malloc(sizeof(*ia), M_IFADDR, M_WAITOK); - if (ia == NULL) - return (ENOBUFS); bzero((caddr_t)ia, sizeof(*ia)); /* Initialize the address and masks */ ia->ia_ifa.ifa_addr = (struct sockaddr *)&ia->ia_addr; diff --git a/sys/netinet6/ip6_output.c b/sys/netinet6/ip6_output.c index 901dddb226d..7fa15e90167 100644 --- a/sys/netinet6/ip6_output.c +++ b/sys/netinet6/ip6_output.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ip6_output.c,v 1.32 2001/04/14 00:30:59 angelos Exp $ */ +/* $OpenBSD: ip6_output.c,v 1.33 2001/05/16 12:53:36 ho Exp $ */ /* $KAME: ip6_output.c,v 1.172 2001/03/25 09:55:56 itojun Exp $ */ /* @@ -1775,8 +1775,6 @@ ip6_setmoptions(optname, im6op, m) im6o = (struct ip6_moptions *) malloc(sizeof(*im6o), M_IPMOPTS, M_WAITOK); - if (im6o == NULL) - return(ENOBUFS); *im6op = im6o; im6o->im6o_multicast_ifp = NULL; im6o->im6o_multicast_hlim = ip6_defmcasthlim; @@ -1940,10 +1938,7 @@ ip6_setmoptions(optname, im6op, m) * address list for the given interface. */ imm = malloc(sizeof(*imm), M_IPMADDR, M_WAITOK); - if (imm == NULL) { - error = ENOBUFS; - break; - } + if ((imm->i6mm_maddr = in6_addmulti(&mreq->ipv6mr_multiaddr, ifp, &error)) == NULL) { free(imm, M_IPMADDR); diff --git a/sys/netipx/ipx.c b/sys/netipx/ipx.c index 1107eb32c58..30a337cb289 100644 --- a/sys/netipx/ipx.c +++ b/sys/netipx/ipx.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ipx.c,v 1.9 2000/01/15 18:52:13 fgsch Exp $ */ +/* $OpenBSD: ipx.c,v 1.10 2001/05/16 12:54:06 ho Exp $ */ /*- * @@ -101,8 +101,6 @@ ipx_control(so, cmd, data, ifp) if (ia == (struct ipx_ifaddr *)NULL) { ia = (struct ipx_ifaddr *) malloc(sizeof(*ia), M_IFADDR, M_WAITOK); - if (ia == (struct ipx_ifaddr *)NULL) - return (ENOBUFS); bzero((caddr_t)ia, sizeof(*ia)); TAILQ_INSERT_TAIL(&ifp->if_addrlist, (struct ifaddr *)ia, ifa_list); diff --git a/sys/netiso/clnp_raw.c b/sys/netiso/clnp_raw.c index 67694c2b68d..287f31d7494 100644 --- a/sys/netiso/clnp_raw.c +++ b/sys/netiso/clnp_raw.c @@ -1,4 +1,4 @@ -/* $OpenBSD: clnp_raw.c,v 1.2 1996/03/04 10:34:59 mickey Exp $ */ +/* $OpenBSD: clnp_raw.c,v 1.3 2001/05/16 12:54:06 ho Exp $ */ /* $NetBSD: clnp_raw.c,v 1.9 1996/02/13 22:08:42 christos Exp $ */ /*- @@ -314,8 +314,6 @@ clnp_usrreq(so, req, m, nam, control) if (rp) panic("rip_attach"); MALLOC(rp, struct rawisopcb *, sizeof *rp, M_PCB, M_WAITOK); - if (rp == 0) - return (ENOBUFS); bzero(rp, sizeof *rp); so->so_pcb = rp; break; diff --git a/sys/netiso/esis.c b/sys/netiso/esis.c index 10b4b86674e..6cf5d0a0409 100644 --- a/sys/netiso/esis.c +++ b/sys/netiso/esis.c @@ -1,4 +1,4 @@ -/* $OpenBSD: esis.c,v 1.4 1996/05/10 12:31:21 deraadt Exp $ */ +/* $OpenBSD: esis.c,v 1.5 2001/05/16 12:54:06 ho Exp $ */ /* $NetBSD: esis.c,v 1.14 1996/05/07 02:45:04 thorpej Exp $ */ /*- @@ -184,13 +184,11 @@ esis_usrreq(so, req, m, nam, control) break; } MALLOC(rp, struct rawcb *, sizeof(*rp), M_PCB, M_WAITOK); - if ((so->so_pcb = rp) != NULL) { - bzero(so->so_pcb, sizeof(*rp)); - LIST_INSERT_HEAD(&esis_pcb, rp, rcb_list); - rp->rcb_socket = so; - error = soreserve(so, esis_sendspace, esis_recvspace); - } else - error = ENOBUFS; + so->so_pcb = rp; + bzero(so->so_pcb, sizeof(*rp)); + LIST_INSERT_HEAD(&esis_pcb, rp, rcb_list); + rp->rcb_socket = so; + error = soreserve(so, esis_sendspace, esis_recvspace); break; case PRU_SEND: diff --git a/sys/netiso/if_cons.c b/sys/netiso/if_cons.c index 99af200fc70..4c7dc72fa03 100644 --- a/sys/netiso/if_cons.c +++ b/sys/netiso/if_cons.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_cons.c,v 1.2 1996/03/04 10:35:21 mickey Exp $ */ +/* $OpenBSD: if_cons.c,v 1.3 2001/05/16 12:54:07 ho Exp $ */ /* $NetBSD: if_cons.c,v 1.7 1996/02/13 22:09:44 christos Exp $ */ /*- @@ -595,8 +595,6 @@ make_partial_x25_packet(isop, lcp) return 0; } MGETHDR(m, M_WAITOK, MT_DATA); - if (m == 0) - return ENOBUFS; buf = mtod(m, caddr_t); ptr = buf; diff --git a/sys/netiso/iso.c b/sys/netiso/iso.c index b9385efee65..c11334c58e9 100644 --- a/sys/netiso/iso.c +++ b/sys/netiso/iso.c @@ -1,4 +1,4 @@ -/* $OpenBSD: iso.c,v 1.3 1996/04/21 22:29:25 deraadt Exp $ */ +/* $OpenBSD: iso.c,v 1.4 2001/05/16 12:54:07 ho Exp $ */ /* $NetBSD: iso.c,v 1.14 1996/04/13 01:34:48 cgd Exp $ */ /*- @@ -488,8 +488,6 @@ iso_control(so, cmd, data, ifp) #endif MALLOC(ia, struct iso_ifaddr *, sizeof(*ia), M_IFADDR, M_WAITOK); - if (ia == 0) - return (ENOBUFS); bzero((caddr_t) ia, sizeof(*ia)); TAILQ_INSERT_TAIL(&iso_ifaddr, ia, ia_list); TAILQ_INSERT_TAIL(&ifp->if_addrlist, (struct ifaddr *) ia, diff --git a/sys/netiso/tp.trans b/sys/netiso/tp.trans index f8ec072e064..7101080e1a0 100644 --- a/sys/netiso/tp.trans +++ b/sys/netiso/tp.trans @@ -1,4 +1,4 @@ -/* $OpenBSD: tp.trans,v 1.2 1996/03/04 10:35:46 mickey Exp $ */ +/* $OpenBSD: tp.trans,v 1.3 2001/05/16 12:54:07 ho Exp $ */ /* $NetBSD: tp.trans,v 1.4 1994/06/29 06:39:55 cgd Exp $ */ /* NEW */ @@ -420,13 +420,11 @@ TP_CRSENT <== TP_CLOSED T_CONN_req $P.tp_ucddata, 0, 0); ENDTRACE data = MCPY($P.tp_ucddata, M_WAIT); - if (data) { - IFDEBUG(D_CONN) - printf("T_CONN_req.trans m_copy cc 0x%x\n", - $P.tp_ucddata); - dump_mbuf(data, "sosnd @ T_CONN_req"); - ENDDEBUG - } + IFDEBUG(D_CONN) + printf("T_CONN_req.trans m_copy cc 0x%x\n", + $P.tp_ucddata); + dump_mbuf(data, "sosnd @ T_CONN_req"); + ENDDEBUG if (error = tp_emit(CR_TPDU_type, $P, 0, 0, data) ) return error; /* driver WON'T change state; will return error */ diff --git a/sys/netiso/tp_driver.c b/sys/netiso/tp_driver.c index 55db1bde56d..c2df0f0ee08 100644 --- a/sys/netiso/tp_driver.c +++ b/sys/netiso/tp_driver.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tp_driver.c,v 1.4 1998/04/04 02:55:57 don Exp $ */ +/* $OpenBSD: tp_driver.c,v 1.5 2001/05/16 12:54:07 ho Exp $ */ /* $NetBSD: tp_driver.c,v 1.8 1996/03/16 23:13:45 christos Exp $ */ #include "tp_states.h" @@ -161,15 +161,13 @@ _Xebec_action(a, e, p) (int) p->tp_flags, p->tp_ucddata, 0, 0); #endif data = MCPY(p->tp_ucddata, M_WAIT); - if (data) { #ifdef ARGO_DEBUG - if (argo_debug[D_CONN]) { - printf("T_CONN_req.trans m_copy cc %p\n", - p->tp_ucddata); - dump_mbuf(data, "sosnd @ T_CONN_req"); - } -#endif + if (argo_debug[D_CONN]) { + printf("T_CONN_req.trans m_copy cc %p\n", + p->tp_ucddata); + dump_mbuf(data, "sosnd @ T_CONN_req"); } +#endif if ((error = tp_emit(CR_TPDU_type, p, 0, 0, data)) != 0) return error; /* driver WON'T change state; * will return error */ diff --git a/sys/netiso/tp_subr.c b/sys/netiso/tp_subr.c index 602b452fc84..73bcff21107 100644 --- a/sys/netiso/tp_subr.c +++ b/sys/netiso/tp_subr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tp_subr.c,v 1.3 1996/04/21 22:29:57 deraadt Exp $ */ +/* $OpenBSD: tp_subr.c,v 1.4 2001/05/16 12:54:07 ho Exp $ */ /* $NetBSD: tp_subr.c,v 1.8 1996/03/16 23:14:00 christos Exp $ */ /*- @@ -721,10 +721,9 @@ tp_packetize(tpcb, m, eotsdu) while (m) { n = m; - if (totlen > maxsize) { - if ((m = m_split(n, maxsize, M_WAIT)) == 0) - panic("tp_packetize"); - } else + if (totlen > maxsize) + m = m_split(n, maxsize, M_WAIT); + else m = 0; totlen -= maxsize; tpsbcheck(tpcb, 5); diff --git a/sys/netiso/tp_usrreq.c b/sys/netiso/tp_usrreq.c index 7c6f03cec22..8b3e4caf987 100644 --- a/sys/netiso/tp_usrreq.c +++ b/sys/netiso/tp_usrreq.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tp_usrreq.c,v 1.3 1996/04/21 22:30:04 deraadt Exp $ */ +/* $OpenBSD: tp_usrreq.c,v 1.4 2001/05/16 12:54:07 ho Exp $ */ /* $NetBSD: tp_usrreq.c,v 1.9 1996/03/16 23:14:06 christos Exp $ */ /*- @@ -329,9 +329,6 @@ tp_sendoob(tpcb, so, xdata, outflags) if (xdata == (struct mbuf *) 0) { /* empty xpd packet */ MGETHDR(xdata, M_WAIT, MT_OOBDATA); - if (xdata == NULL) { - return ENOBUFS; - } xdata->m_len = 0; xdata->m_pkthdr.len = 0; } diff --git a/sys/netnatm/natm.c b/sys/netnatm/natm.c index bff8f2468ae..e3f6a5f8f88 100644 --- a/sys/netnatm/natm.c +++ b/sys/netnatm/natm.c @@ -1,4 +1,4 @@ -/* $OpenBSD: natm.c,v 1.5 1997/10/11 01:32:08 angelos Exp $ */ +/* $OpenBSD: natm.c,v 1.6 2001/05/16 12:54:08 ho Exp $ */ /* * @@ -246,10 +246,6 @@ struct proc *p; */ M_PREPEND(m, sizeof(*aph), M_WAITOK); - if (m == NULL) { - error = ENOBUFS; - break; - } aph = mtod(m, struct atm_pseudohdr *); ATM_PH_VPI(aph) = npcb->npcb_vpi; ATM_PH_SETVCI(aph, npcb->npcb_vci); diff --git a/sys/netnatm/natm_pcb.c b/sys/netnatm/natm_pcb.c index 74c06708f89..06bd49a5a57 100644 --- a/sys/netnatm/natm_pcb.c +++ b/sys/netnatm/natm_pcb.c @@ -1,4 +1,4 @@ -/* $OpenBSD: natm_pcb.c,v 1.3 1996/11/09 03:29:00 chuck Exp $ */ +/* $OpenBSD: natm_pcb.c,v 1.4 2001/05/16 12:54:08 ho Exp $ */ /* * @@ -67,10 +67,6 @@ int wait; MALLOC(npcb, struct natmpcb *, sizeof(*npcb), M_PCB, wait); -#ifdef DIAGNOSTIC - if (wait == M_WAITOK && npcb == NULL) panic("npcb_alloc: malloc didn't wait"); -#endif - if (npcb) { bzero(npcb, sizeof(*npcb)); npcb->npcb_flags = NPCB_FREE; diff --git a/sys/netns/ns.c b/sys/netns/ns.c index 338c3701394..a72458f2f5e 100644 --- a/sys/netns/ns.c +++ b/sys/netns/ns.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ns.c,v 1.2 1996/03/04 08:20:22 niklas Exp $ */ +/* $OpenBSD: ns.c,v 1.3 2001/05/16 12:54:07 ho Exp $ */ /* $NetBSD: ns.c,v 1.10 1996/02/13 22:13:49 christos Exp $ */ /* @@ -131,8 +131,6 @@ ns_control(so, cmd, data, ifp) if (ia == 0) { ia = (struct ns_ifaddr *) malloc(sizeof *ia, M_IFADDR, M_WAITOK); - if (ia == 0) - return (ENOBUFS); bzero((caddr_t)ia, sizeof(*ia)); TAILQ_INSERT_TAIL(&ns_ifaddr, ia, ia_list); TAILQ_INSERT_TAIL(&ifp->if_addrlist, (struct ifaddr *)ia, diff --git a/sys/nfs/krpc_subr.c b/sys/nfs/krpc_subr.c index fb8ec0cb94b..817f95010e6 100644 --- a/sys/nfs/krpc_subr.c +++ b/sys/nfs/krpc_subr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: krpc_subr.c,v 1.9 1998/02/28 14:03:08 deraadt Exp $ */ +/* $OpenBSD: krpc_subr.c,v 1.10 2001/05/16 12:48:32 ho Exp $ */ /* $NetBSD: krpc_subr.c,v 1.12.4.1 1996/06/07 00:52:26 cgd Exp $ */ /* @@ -153,8 +153,6 @@ krpc_portmap(sin, prog, vers, portp) } m = m_get(M_WAIT, MT_DATA); - if (m == NULL) - return ENOBUFS; sdata = mtod(m, struct sdata *); m->m_len = sizeof(*sdata); @@ -204,6 +202,7 @@ krpc_call(sa, prog, vers, func, data, from_p) static u_int32_t xid = 0; u_int32_t newxid; int *ip; + struct timeval *tv; /* * Validate address family. @@ -223,18 +222,12 @@ krpc_call(sa, prog, vers, func, data, from_p) goto out; m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) { - error = ENOBUFS; + tv = mtod(m, struct timeval *); + m->m_len = sizeof(*tv); + tv->tv_sec = 1; + tv->tv_usec = 0; + if ((error = sosetopt(so, SOL_SOCKET, SO_RCVTIMEO, m))) goto out; - } else { - struct timeval *tv; - tv = mtod(m, struct timeval *); - m->m_len = sizeof(*tv); - tv->tv_sec = 1; - tv->tv_usec = 0; - if ((error = sosetopt(so, SOL_SOCKET, SO_RCVTIMEO, m))) - goto out; - } /* * Enable broadcast if necessary. @@ -242,10 +235,6 @@ krpc_call(sa, prog, vers, func, data, from_p) if (from_p) { int32_t *on; m = m_get(M_WAIT, MT_SOOPTS); - if (m == NULL) { - error = ENOBUFS; - goto out; - } on = mtod(m, int32_t *); m->m_len = sizeof(*on); *on = 1; @@ -297,10 +286,6 @@ krpc_call(sa, prog, vers, func, data, from_p) * Setup socket address for the server. */ nam = m_get(M_WAIT, MT_SONAME); - if (nam == NULL) { - error = ENOBUFS; - goto out; - } sin = mtod(nam, struct sockaddr_in *); bcopy((caddr_t)sa, (caddr_t)sin, (nam->m_len = sa->sin_len)); diff --git a/sys/nfs/nfs_vfsops.c b/sys/nfs/nfs_vfsops.c index 3640ed142e4..3a43c37a635 100644 --- a/sys/nfs/nfs_vfsops.c +++ b/sys/nfs/nfs_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_vfsops.c,v 1.32 2001/02/20 01:50:11 assar Exp $ */ +/* $OpenBSD: nfs_vfsops.c,v 1.33 2001/05/16 12:48:32 ho Exp $ */ /* $NetBSD: nfs_vfsops.c,v 1.46.4.1 1996/05/25 22:40:35 fvdl Exp $ */ /* @@ -418,8 +418,6 @@ nfs_mount_diskless(ndmntp, mntname, mntflag, vpp) /* Get mbuf for server sockaddr. */ m = m_get(M_WAIT, MT_SONAME); - if (m == NULL) - panic("nfs_mountroot: mget soname for %s", mntname); bcopy((caddr_t)args.addr, mtod(m, caddr_t), (m->m_len = args.addr->sa_len)); diff --git a/sys/vm/device_pager.c b/sys/vm/device_pager.c index 2cb8603f19d..34423923334 100644 --- a/sys/vm/device_pager.c +++ b/sys/vm/device_pager.c @@ -1,4 +1,4 @@ -/* $OpenBSD: device_pager.c,v 1.5 1997/11/10 09:09:49 niklas Exp $ */ +/* $OpenBSD: device_pager.c,v 1.6 2001/05/16 12:54:34 ho Exp $ */ /* $NetBSD: device_pager.c,v 1.24 1997/01/03 18:03:14 mrg Exp $ */ /* @@ -165,13 +165,7 @@ top: * Allocate and initialize pager structs */ pager = (vm_pager_t)malloc(sizeof *pager, M_VMPAGER, M_WAITOK); - if (pager == NULL) - return(NULL); devp = (dev_pager_t)malloc(sizeof *devp, M_VMPGDATA, M_WAITOK); - if (devp == NULL) { - free((caddr_t)pager, M_VMPAGER); - return(NULL); - } pager->pg_handle = handle; pager->pg_ops = &devicepagerops; pager->pg_type = PG_DEVICE; diff --git a/sys/vm/vm_map.c b/sys/vm/vm_map.c index cc315240115..d3dac7a832a 100644 --- a/sys/vm/vm_map.c +++ b/sys/vm/vm_map.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vm_map.c,v 1.19 2001/04/06 23:41:02 art Exp $ */ +/* $OpenBSD: vm_map.c,v 1.20 2001/05/16 12:54:34 ho Exp $ */ /* $NetBSD: vm_map.c,v 1.23 1996/02/10 00:08:08 christos Exp $ */ /* @@ -378,8 +378,6 @@ vm_map_entry_create(map) } else { MALLOC(entry, vm_map_entry_t, sizeof(struct vm_map_entry), M_VMMAPENT, M_WAITOK); - if (entry == NULL) - panic("vm_map_entry_create: couldn't alloc pageable map entry"); } } else { s = splimp(); diff --git a/sys/vm/vnode_pager.c b/sys/vm/vnode_pager.c index 3179af43f71..d1c885fb65c 100644 --- a/sys/vm/vnode_pager.c +++ b/sys/vm/vnode_pager.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vnode_pager.c,v 1.7 1997/12/10 02:36:54 csapuntz Exp $ */ +/* $OpenBSD: vnode_pager.c,v 1.8 2001/05/16 12:54:34 ho Exp $ */ /* $NetBSD: vnode_pager.c,v 1.19 1996/03/16 23:15:27 christos Exp $ */ /* @@ -148,13 +148,7 @@ vnode_pager_alloc(handle, size, prot, foff) * Allocate pager structures */ pager = (vm_pager_t)malloc(sizeof *pager, M_VMPAGER, M_WAITOK); - if (pager == NULL) - return(NULL); vnp = (vn_pager_t)malloc(sizeof *vnp, M_VMPGDATA, M_WAITOK); - if (vnp == NULL) { - free((caddr_t)pager, M_VMPAGER); - return(NULL); - } /* * And an object of the appropriate size */ |