kernel: Remove NULL checks after kmalloc() with M_WAITOK.
authorSascha Wildner <saw@online.de>
Sun, 8 Apr 2012 23:28:51 +0000 (01:28 +0200)
committerSascha Wildner <saw@online.de>
Sun, 8 Apr 2012 23:28:51 +0000 (01:28 +0200)
15 files changed:
sys/dev/disk/isp/isp_freebsd.c
sys/dev/disk/sili/sili.c
sys/dev/netif/iwi/if_iwi.c
sys/dev/netif/lgue/if_lgue.c
sys/dev/netif/mxge/if_mxge.c
sys/dev/raid/mps/mps_sas.c
sys/dev/sound/pcm/feeder_fmt.c
sys/kern/link_elf_obj.c
sys/net/altq/altq_hfsc.c
sys/net/pf/pf_ioctl.c
sys/netgraph7/netflow/netflow.c
sys/netgraph7/ng_device.c
sys/netgraph7/ng_tty.c
sys/opencrypto/cryptosoft.c
sys/opencrypto/deflate.c

index ad59b93..1d34355 100644 (file)
@@ -3518,24 +3518,12 @@ isp_target_thread(ispsoftc_t *isp, int chan)
                        disk_size = 50 << 20;
                }
                disk_data = kmalloc(disk_size, M_ISPTARG, M_WAITOK | M_ZERO);
-               if (disk_data == NULL) {
-                       isp_prt(isp, ISP_LOGERR, "%s: could not allocate disk data", __func__);
-                       goto out;
-               }
                isp_prt(isp, ISP_LOGINFO, "allocated a %ju MiB disk", (uintmax_t) (disk_size >> 20));
        }
        junk_data = kmalloc(JUNK_SIZE, M_ISPTARG, M_WAITOK | M_ZERO);
-       if (junk_data == NULL) {
-               isp_prt(isp, ISP_LOGERR, "%s: could not allocate junk", __func__);
-               goto out;
-       }
 
 
        softc = kmalloc(sizeof (*softc), M_ISPTARG, M_WAITOK | M_ZERO);
-       if (softc == NULL) {
-               isp_prt(isp, ISP_LOGERR, "%s: could not allocate softc", __func__);
-               goto out;
-       }
        TAILQ_INIT(&softc->work_queue);
        TAILQ_INIT(&softc->rework_queue);
        TAILQ_INIT(&softc->running_queue);
index 6809bf5..23cc996 100644 (file)
@@ -210,12 +210,6 @@ sili_port_alloc(struct sili_softc *sc, u_int port)
         */
        ap->ap_ccbs = kmalloc(sizeof(struct sili_ccb) * sc->sc_ncmds, M_DEVBUF,
                              M_WAITOK | M_ZERO);
-       if (ap->ap_ccbs == NULL) {
-               device_printf(sc->sc_dev,
-                             "unable to allocate command list for port %d\n",
-                             port);
-               goto freeport;
-       }
 
        /*
         * Most structures are in the port BAR.  Assign convenient
index 1342879..8876bf7 100644 (file)
@@ -660,11 +660,6 @@ iwi_alloc_tx_ring(struct iwi_softc *sc, struct iwi_tx_ring *ring, int count,
 
        ring->data = kmalloc(count * sizeof (struct iwi_tx_data), M_DEVBUF,
            M_WAITOK | M_ZERO);
-       if (ring->data == NULL) {
-               device_printf(sc->sc_dev, "could not allocate soft data\n");
-               error = ENOMEM;
-               goto fail;
-       }
 
        error = bus_dma_tag_create(NULL, 1, 0, BUS_SPACE_MAXADDR_32BIT,
            BUS_SPACE_MAXADDR, NULL, NULL, MCLBYTES, IWI_MAX_NSEG,
@@ -768,11 +763,6 @@ iwi_alloc_rx_ring(struct iwi_softc *sc, struct iwi_rx_ring *ring, int count)
 
        ring->data = kmalloc(count * sizeof (struct iwi_rx_data), M_DEVBUF,
            M_WAITOK | M_ZERO);
-       if (ring->data == NULL) {
-               device_printf(sc->sc_dev, "could not allocate soft data\n");
-               error = ENOMEM;
-               goto fail;
-       }
 
        error = bus_dma_tag_create(NULL, 1, 0, BUS_SPACE_MAXADDR_32BIT,
            BUS_SPACE_MAXADDR, NULL, NULL, MCLBYTES, 1, MCLBYTES,
index 0b8780d..6e1a211 100644 (file)
@@ -677,10 +677,6 @@ lgue_init(void *xsc)
                }
        }
        sc->lgue_tx_buf = kmalloc(LGUE_BUFSZ, M_USBDEV, M_WAITOK);
-       if (sc->lgue_tx_buf == NULL) {
-               if_printf(ifp, "tx internal buffer allocate failed\n");
-               return;
-       }
 
        if (sc->lgue_rx_xfer == NULL) {
                sc->lgue_rx_xfer = usbd_alloc_xfer(sc->lgue_udev);
@@ -690,10 +686,6 @@ lgue_init(void *xsc)
                }
        }
        sc->lgue_rx_buf = kmalloc(LGUE_BUFSZ, M_USBDEV, M_WAITOK);
-       if (sc->lgue_rx_buf == NULL) {
-               if_printf(ifp, "rx internal buffer allocate failed\n");
-               return;
-       }
 
        /* Create INTR buf */
        if (sc->lgue_intr_xfer == NULL) {
index b444890..c80a6b4 100644 (file)
@@ -3186,24 +3186,16 @@ mxge_alloc_slice_rings(struct mxge_slice_state *ss, int rx_ring_entries,
        /* allocate the rx shadow rings */
        bytes = rx_ring_entries * sizeof (*ss->rx_small.shadow);
        ss->rx_small.shadow = kmalloc(bytes, M_DEVBUF, M_ZERO|M_WAITOK);
-       if (ss->rx_small.shadow == NULL)
-               return err;;
 
        bytes = rx_ring_entries * sizeof (*ss->rx_big.shadow);
        ss->rx_big.shadow = kmalloc(bytes, M_DEVBUF, M_ZERO|M_WAITOK);
-       if (ss->rx_big.shadow == NULL)
-               return err;;
 
        /* allocate the rx host info rings */
        bytes = rx_ring_entries * sizeof (*ss->rx_small.info);
        ss->rx_small.info = kmalloc(bytes, M_DEVBUF, M_ZERO|M_WAITOK);
-       if (ss->rx_small.info == NULL)
-               return err;;
 
        bytes = rx_ring_entries * sizeof (*ss->rx_big.info);
        ss->rx_big.info = kmalloc(bytes, M_DEVBUF, M_ZERO|M_WAITOK);
-       if (ss->rx_big.info == NULL)
-               return err;;
 
        /* allocate the rx busdma resources */
        err = bus_dma_tag_create(sc->parent_dmat,       /* parent */
@@ -3298,8 +3290,6 @@ mxge_alloc_slice_rings(struct mxge_slice_state *ss, int rx_ring_entries,
        bytes = 8 + 
                sizeof (*ss->tx.req_list) * (ss->tx.max_desc + 4);
        ss->tx.req_bytes = kmalloc(bytes, M_DEVBUF, M_WAITOK);
-       if (ss->tx.req_bytes == NULL)
-               return err;;
        /* ensure req_list entries are aligned to 8 bytes */
        ss->tx.req_list = (mcp_kreq_ether_send_t *)
                ((unsigned long)(ss->tx.req_bytes + 7) & ~7UL);
@@ -3314,8 +3304,6 @@ mxge_alloc_slice_rings(struct mxge_slice_state *ss, int rx_ring_entries,
        /* allocate the tx host info ring */
        bytes = tx_ring_entries * sizeof (*ss->tx.info);
        ss->tx.info = kmalloc(bytes, M_DEVBUF, M_ZERO|M_WAITOK);
-       if (ss->tx.info == NULL)
-               return err;;
        
        /* allocate the tx busdma resources */
        err = bus_dma_tag_create(sc->parent_dmat,       /* parent */
index 889b7c5..d58eafe 100644 (file)
@@ -3003,15 +3003,6 @@ mpssas_check_eedp(struct mpssas_softc *sassc)
                                if (!found_lun) {
                                        lun = kmalloc(sizeof(struct mpssas_lun),
                                            M_MPT2, M_WAITOK | M_ZERO);
-                                       if (lun == NULL) {
-                                               mps_dprint(sc, MPS_FAULT,
-                                                   "Unable to alloc LUN for "
-                                                   "EEDP support.\n");
-                                               kfree(rcap_buf, M_MPT2);
-                                               xpt_free_path(ccb->ccb_h.path);
-                                               xpt_free_ccb(ccb);
-                                               return;
-                                       }
                                        lun->lun_id = lunid;
                                        SLIST_INSERT_HEAD(&target->luns, lun,
                                            lun_link);
index 0650a6a..9579b6e 100644 (file)
@@ -325,8 +325,6 @@ static int
 feed_common_init(struct pcm_feeder *f)
 {
        f->data = kmalloc(FEEDBUFSZ, M_FMTFEEDER, M_WAITOK|M_ZERO);
-       if (f->data == NULL)
-               return ENOMEM;
        return 0;
 }
 
index b7bd1db..d4a23f8 100644 (file)
@@ -573,10 +573,6 @@ link_elf_obj_load_file(const char *filename, linker_file_t * result)
        /* Allocate space for and load the symbol table */
        ef->ddbsymcnt = shdr[symtabindex].sh_size / sizeof(Elf_Sym);
        ef->ddbsymtab = kmalloc(shdr[symtabindex].sh_size, M_LINKER, M_WAITOK);
-       if (ef->ddbsymtab == NULL) {
-               error = ENOMEM;
-               goto out;
-       }
        error = vn_rdwr(UIO_READ, vp, (void *)ef->ddbsymtab,
                        shdr[symtabindex].sh_size, shdr[symtabindex].sh_offset,
                        UIO_SYSSPACE, IO_NODELOCKED, p->p_ucred, &resid);
@@ -591,10 +587,6 @@ link_elf_obj_load_file(const char *filename, linker_file_t * result)
        /* Allocate space for and load the symbol strings */
        ef->ddbstrcnt = shdr[symstrindex].sh_size;
        ef->ddbstrtab = kmalloc(shdr[symstrindex].sh_size, M_LINKER, M_WAITOK);
-       if (ef->ddbstrtab == NULL) {
-               error = ENOMEM;
-               goto out;
-       }
        error = vn_rdwr(UIO_READ, vp, ef->ddbstrtab,
                        shdr[symstrindex].sh_size, shdr[symstrindex].sh_offset,
                        UIO_SYSSPACE, IO_NODELOCKED, p->p_ucred, &resid);
@@ -612,10 +604,6 @@ link_elf_obj_load_file(const char *filename, linker_file_t * result)
                ef->shstrcnt = shdr[shstrindex].sh_size;
                ef->shstrtab = kmalloc(shdr[shstrindex].sh_size, M_LINKER,
                                       M_WAITOK);
-               if (ef->shstrtab == NULL) {
-                       error = ENOMEM;
-                       goto out;
-               }
                error = vn_rdwr(UIO_READ, vp, ef->shstrtab,
                                shdr[shstrindex].sh_size, shdr[shstrindex].sh_offset,
                                UIO_SYSSPACE, IO_NODELOCKED, p->p_ucred, &resid);
index 9b34a96..2c3d69a 100644 (file)
@@ -450,15 +450,11 @@ hfsc_class_create(struct hfsc_if *hif, struct service_curve *rsc,
        }
        if (fsc != NULL && (fsc->m1 != 0 || fsc->m2 != 0)) {
                cl->cl_fsc = kmalloc(sizeof(*cl->cl_fsc), M_ALTQ, M_WAITOK);
-               if (cl->cl_fsc == NULL)
-                       goto err_ret;
                sc2isc(fsc, cl->cl_fsc);
                rtsc_init(&cl->cl_virtual, cl->cl_fsc, 0, 0);
        }
        if (usc != NULL && (usc->m1 != 0 || usc->m2 != 0)) {
                cl->cl_usc = kmalloc(sizeof(*cl->cl_usc), M_ALTQ, M_WAITOK);
-               if (cl->cl_usc == NULL)
-                       goto err_ret;
                sc2isc(usc, cl->cl_usc);
                rtsc_init(&cl->cl_ulimit, cl->cl_usc, 0, 0);
        }
index 8fb6914..b74fb19 100644 (file)
@@ -908,9 +908,6 @@ pf_setup_pfsync_matching(struct pf_ruleset *rs)
                            kmalloc(sizeof(caddr_t) *
                                    rs->rules[rs_cnt].inactive.rcount,
                                    M_TEMP, M_WAITOK);
-
-                       if (!rs->rules[rs_cnt].inactive.ptr_array)
-                               return (ENOMEM);
                }
 
                TAILQ_FOREACH(rule, rs->rules[rs_cnt].inactive.ptr,
index 70f68cf..9741951 100644 (file)
@@ -26,7 +26,6 @@
  *
  * $SourceForge: netflow.c,v 1.41 2004/09/05 11:41:10 glebius Exp $
  * $FreeBSD: src/sys/netgraph/netflow/netflow.c,v 1.29 2008/05/09 23:02:57 julian Exp $
- * $DragonFly: src/sys/netgraph7/netflow/netflow.c,v 1.2 2008/06/26 23:05:40 dillon Exp $
  */
 
 #include <sys/param.h>
@@ -333,11 +332,6 @@ ng_netflow_cache_init(priv_p priv)
        priv->hash = kmalloc(NBUCKETS * sizeof(struct flow_hash_entry),
                             M_NETFLOW_HASH, M_WAITOK | M_ZERO);
 
-       if (priv->hash == NULL) {
-               uma_zdestroy(priv->zone);
-               return (ENOMEM);
-       }
-
        /* Initialize hash. */
        for (i = 0, hsh = priv->hash; i < NBUCKETS; i++, hsh++) {
                mtx_init(&hsh->mtx, "hash mutex", NULL, MTX_DEF);
index 58f7487..502a1fe 100644 (file)
@@ -28,7 +28,6 @@
  * netgraph node.
  *
  * $FreeBSD: src/sys/netgraph/ng_device.c,v 1.22 2006/11/02 17:37:21 andre Exp $
- * $DragonFly: src/sys/netgraph7/ng_device.c,v 1.2 2008/06/26 23:05:35 dillon Exp $
  *
  */
 
@@ -164,8 +163,6 @@ ng_device_constructor(node_p node)
        DBG;
 
        priv = kmalloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO);
-       if (priv == NULL)
-               return (ENOMEM);
 
        /* Allocate unit number */
        priv->unit = alloc_unr(ngd_unit);
index fc4beeb..16523f1 100644 (file)
@@ -38,7 +38,6 @@
  * Author: Archie Cobbs <archie@freebsd.org>
  *
  * $FreeBSD: src/sys/netgraph/ng_tty.c,v 1.37 2006/11/06 13:42:03 rwatson Exp $
- * $DragonFly: src/sys/netgraph7/ng_tty.c,v 1.2 2008/06/26 23:05:35 dillon Exp $
  * $Whistle: ng_tty.c,v 1.21 1999/11/01 09:24:52 julian Exp $
  */
 
@@ -196,8 +195,6 @@ ngt_open(struct cdev *dev, struct tty *tp)
 
        /* Initialize private struct */
        sc = kmalloc(sizeof(*sc), M_NETGRAPH, M_WAITOK | M_ZERO);
-       if (sc == NULL)
-               return (ENOMEM);
 
        lwkt_gettoken(&tty_token);
        sc->tp = tp;
index f2b735c..7439a7c 100644 (file)
@@ -936,17 +936,9 @@ swcr_newsession(device_t dev, u_int32_t *sid, struct cryptoini *cri)
                authcommon:
                        (*swd)->sw_ictx = kmalloc(axf->ctxsize, M_CRYPTO_DATA,
                                                  M_WAITOK);
-                       if ((*swd)->sw_ictx == NULL) {
-                               swcr_freesession_slot(&swd_base, 0);
-                               return ENOBUFS;
-                       }
        
                        (*swd)->sw_octx = kmalloc(axf->ctxsize, M_CRYPTO_DATA,
                                                  M_WAITOK);
-                       if ((*swd)->sw_octx == NULL) {
-                               swcr_freesession_slot(&swd_base, 0);
-                               return ENOBUFS;
-                       }
        
                        if (cri->cri_key != NULL) {
                                swcr_authprepare(axf, *swd, cri->cri_key,
@@ -966,17 +958,9 @@ swcr_newsession(device_t dev, u_int32_t *sid, struct cryptoini *cri)
                auth2common:
                        (*swd)->sw_ictx = kmalloc(axf->ctxsize, M_CRYPTO_DATA,
                                                  M_WAITOK);
-                       if ((*swd)->sw_ictx == NULL) {
-                               swcr_freesession_slot(&swd_base, 0);
-                               return ENOBUFS;
-                       }
        
                        (*swd)->sw_octx = kmalloc(cri->cri_klen / 8,
                                                  M_CRYPTO_DATA, M_WAITOK);
-                       if ((*swd)->sw_octx == NULL) {
-                               swcr_freesession_slot(&swd_base, 0);
-                               return ENOBUFS;
-                       }
        
                        /* Store the key so we can "append" it to the payload */
                        if (cri->cri_key != NULL) {
@@ -997,10 +981,6 @@ swcr_newsession(device_t dev, u_int32_t *sid, struct cryptoini *cri)
                auth3common:
                        (*swd)->sw_ictx = kmalloc(axf->ctxsize, M_CRYPTO_DATA,
                                                  M_WAITOK);
-                       if ((*swd)->sw_ictx == NULL) {
-                               swcr_freesession_slot(&swd_base, 0);
-                               return ENOBUFS;
-                       }
 
                        axf->Init((*swd)->sw_ictx);
                        (*swd)->sw_mlen = cri->cri_mlen;
index ca63966..0b74623 100644 (file)
@@ -74,8 +74,6 @@ deflate_global(u_int8_t *data, u_int32_t size, int decomp, u_int8_t **out)
 
        if (!decomp) {
                buf[i].out = kmalloc((u_long) size, M_CRYPTO_DATA, M_WAITOK);
-               if (buf[i].out == NULL)
-                       goto bad;
                buf[i].size = size;
                buf[i].flag = 1;
                i++;
@@ -89,8 +87,6 @@ deflate_global(u_int8_t *data, u_int32_t size, int decomp, u_int8_t **out)
 
                buf[i].out = kmalloc((u_long) (size * 4),
                                     M_CRYPTO_DATA, M_WAITOK);
-               if (buf[i].out == NULL)
-                       goto bad;
                buf[i].size = size * 4;
                buf[i].flag = 1;
                i++;
@@ -116,8 +112,6 @@ deflate_global(u_int8_t *data, u_int32_t size, int decomp, u_int8_t **out)
                        /* we need more output space, allocate size */
                        buf[i].out = kmalloc((u_long) size,
                                             M_CRYPTO_DATA, M_WAITOK);
-                       if (buf[i].out == NULL)
-                               goto bad;
                        zbuf.next_out = buf[i].out;
                        buf[i].size = size;
                        buf[i].flag = 1;
@@ -131,8 +125,6 @@ end:
        result = count = zbuf.total_out;
 
        *out = kmalloc((u_long) result, M_CRYPTO_DATA, M_WAITOK);
-       if (*out == NULL)
-               goto bad;
        if (decomp)
                inflateEnd(&zbuf);
        else