9ee0be05dc
Coverity (1792) spotted a possibly uninitialized return value in case of
kmalloc() failure:
1116 static int hisax_cs_setup(int cardnr, struct IsdnCard *card,
1117 struct IsdnCardState *cs)
1119 int ret;
1120
1121 if (!(cs->rcvbuf = kmalloc(MAX_DFRAME_LEN_L1, GFP_ATOMIC))) {
1122 printk(KERN_WARNING "HiSax: No memory for isac rcvbuf\n");
1123 ll_unload(cs);
1124 goto outf_cs;
...
1165 outf_cs:
1166 kfree(cs);
1167 card->cs = NULL;
1168 return ret;
The straightforward solution would be to just add the missing
initialization but hardcoding the return value in the out_cs branch
(only taken on failure) seems to work just as well and it allows killing
a couple of other lines too.
Signed-off-by: Florin Malita <fmalita@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
|
||
|---|---|---|
| .. | ||
| Kconfig | ||
| Makefile | ||
| amd7930_fn.c | ||
| amd7930_fn.h | ||
| arcofi.c | ||
| arcofi.h | ||
| asuscom.c | ||
| avm_a1.c | ||
| avm_a1p.c | ||
| avm_pci.c | ||
| avma1_cs.c | ||
| bkm_a4t.c | ||
| bkm_a8.c | ||
| bkm_ax.h | ||
| callc.c | ||
| config.c | ||
| diva.c | ||
| elsa.c | ||
| elsa_cs.c | ||
| elsa_ser.c | ||
| enternow_pci.c | ||
| fsm.c | ||
| fsm.h | ||
| gazel.c | ||
| hfc4s8s_l1.c | ||
| hfc4s8s_l1.h | ||
| hfc_2bds0.c | ||
| hfc_2bds0.h | ||
| hfc_2bs0.c | ||
| hfc_2bs0.h | ||
| hfc_pci.c | ||
| hfc_pci.h | ||
| hfc_sx.c | ||
| hfc_sx.h | ||
| hfc_usb.c | ||
| hfc_usb.h | ||
| hfcscard.c | ||
| hisax.h | ||
| hisax_cfg.h | ||
| hisax_debug.h | ||
| hisax_fcpcipnp.c | ||
| hisax_fcpcipnp.h | ||
| hisax_if.h | ||
| hisax_isac.c | ||
| hisax_isac.h | ||
| hscx.c | ||
| hscx.h | ||
| hscx_irq.c | ||
| icc.c | ||
| icc.h | ||
| ipac.h | ||
| ipacx.c | ||
| ipacx.h | ||
| isac.c | ||
| isac.h | ||
| isar.c | ||
| isar.h | ||
| isdnhdlc.c | ||
| isdnhdlc.h | ||
| isdnl1.c | ||
| isdnl1.h | ||
| isdnl2.c | ||
| isdnl2.h | ||
| isdnl3.c | ||
| isdnl3.h | ||
| isurf.c | ||
| ix1_micro.c | ||
| jade.c | ||
| jade.h | ||
| jade_irq.c | ||
| l3_1tr6.c | ||
| l3_1tr6.h | ||
| l3dss1.c | ||
| l3dss1.h | ||
| l3ni1.c | ||
| l3ni1.h | ||
| lmgr.c | ||
| mic.c | ||
| netjet.c | ||
| netjet.h | ||
| niccy.c | ||
| nj_s.c | ||
| nj_u.c | ||
| q931.c | ||
| s0box.c | ||
| saphir.c | ||
| sedlbauer.c | ||
| sedlbauer_cs.c | ||
| sportster.c | ||
| st5481.h | ||
| st5481_b.c | ||
| st5481_d.c | ||
| st5481_init.c | ||
| st5481_usb.c | ||
| tei.c | ||
| teleint.c | ||
| teles0.c | ||
| teles3.c | ||
| teles_cs.c | ||
| telespci.c | ||
| w6692.c | ||
| w6692.h | ||