Florin Malita
9ee0be05dc
ISDN HiSax: uninitialized return in hisax_cs_setup
...
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>
2007-07-18 18:17:23 -04:00
..
2006-11-22 14:57:56 +00:00
2006-10-03 23:01:26 +02:00
2005-04-16 15:20:36 -07:00
2005-04-16 15:20:36 -07:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2007-02-14 08:09:54 -08:00
2007-07-17 17:24:17 -04:00
2007-07-11 16:02:10 -07:00
2005-04-16 15:20:36 -07:00
2005-06-25 16:25:07 -07:00
2007-07-18 18:17:23 -04:00
2006-12-08 08:29:01 -08:00
2007-02-14 08:09:54 -08:00
2007-04-25 22:28:23 -07:00
2006-10-05 15:10:12 +01:00
2007-07-17 17:23:22 -04:00
2006-12-08 08:29:01 -08:00
2005-04-16 15:20:36 -07:00
2006-10-05 15:10:12 +01:00
2007-02-12 09:48:35 -08:00
2006-11-30 05:24:39 +01:00
2006-11-22 14:57:56 +00:00
2005-04-16 15:20:36 -07:00
2006-07-10 13:24:15 -07:00
2005-04-16 15:20:36 -07:00
2007-07-17 17:26:24 -04:00
2005-06-25 16:25:07 -07:00
2006-11-22 14:57:56 +00:00
2005-06-25 16:25:07 -07:00
2007-05-23 20:14:14 -07:00
2007-02-12 09:48:35 -08:00
2006-10-05 15:10:12 +01:00
2005-04-16 15:20:36 -07:00
2006-06-30 19:25:36 +02:00
2007-05-23 20:14:14 -07:00
2006-01-08 20:14:07 -08:00
2005-04-16 15:20:36 -07:00
2006-12-08 08:29:01 -08:00
2005-04-16 15:20:36 -07:00
2007-02-12 09:48:34 -08:00
2006-01-14 18:27:06 -08:00
2005-11-07 07:54:02 -08:00
2005-04-16 15:20:36 -07:00
2006-11-22 14:57:56 +00:00
2006-07-10 13:24:15 -07:00
2005-04-16 15:20:36 -07:00
2006-07-10 13:24:15 -07:00
2005-04-16 15:20:36 -07:00
2006-11-22 14:57:56 +00:00
2005-04-16 15:20:36 -07:00
2007-03-27 09:05:15 -07:00
2005-04-16 15:20:36 -07:00
2006-12-08 08:28:40 -08:00
2006-12-08 08:28:40 -08:00
2006-11-22 14:57:56 +00:00
2007-02-12 09:48:34 -08:00
2007-04-25 22:28:23 -07:00
2005-04-16 15:20:36 -07:00
2007-02-12 09:48:34 -08:00
2007-02-12 09:48:34 -08:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2006-01-14 18:27:06 -08:00
2006-07-10 13:24:15 -07:00
2005-06-25 16:25:07 -07:00
2007-02-12 09:48:34 -08:00
2005-06-25 16:25:07 -07:00
2005-04-16 15:20:36 -07:00
2006-06-30 19:25:36 +02:00
2005-04-16 15:20:36 -07:00
2005-06-25 16:25:07 -07:00
2005-04-16 15:20:36 -07:00
2005-04-16 15:20:36 -07:00
2007-02-12 09:48:34 -08:00
2006-10-05 15:10:12 +01:00
2007-05-02 19:02:35 -07:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2007-07-17 17:18:29 -04:00
2007-07-17 17:21:37 -04:00
2006-06-26 09:58:23 -07:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2007-02-14 08:09:54 -08:00
2007-07-17 10:23:05 -07:00
2006-10-05 15:10:12 +01:00
2006-12-08 08:28:40 -08:00
2006-12-08 08:29:01 -08:00
2007-05-23 20:14:14 -07:00
2007-05-23 20:14:14 -07:00
2006-06-30 19:25:36 +02:00
2005-06-25 16:25:07 -07:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2006-10-05 15:10:12 +01:00
2007-02-14 08:09:54 -08:00
2006-10-05 15:10:12 +01:00
2006-11-22 14:57:56 +00:00
2005-04-16 15:20:36 -07:00