Skip to contentSkip to conversation

: Return proper error codes in tcf_gact_init

PATCH In review Patrick McHardy 1 participant 1 message Dec 30, 2004

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 8/17 : Return proper error codes in tcf_gact_init

Return proper error codes in tcf_gact_init.

# This is a BitKeeper generated diff -Nru style patch. #
# ChangeSet
# 2004/12/30 02:29:53+01:00 [email protected]
# [PKT_SCHED]: Return proper error codes in tcf_gact_init
#
# Signed-off-by: Patrick McHardy <[email protected]>
#

# net/sched/gact.c
#   2004/12/30 02:29:47+01:00 [email protected] +4 -7
#   [PKT_SCHED]: Return proper error codes in tcf_gact_init
#   
#   Signed-off-by: Patrick McHardy <[email protected]>
# 
diff -Nru a/net/sched/gact.c b/net/sched/gact.c
--- a/net/sched/gact.c	2004-12-30 04:01:28 +01:00
+++ b/net/sched/gact.c	2004-12-30 04:01:28 +01:00
@@ -83,12 +83,9 @@
 	int ret = 0;

if (rtattr_parse(tb, TCA_GACT_MAX, RTA_DATA(rta), RTA_PAYLOAD(rta)) < 0)
- return -1;
-
- if (tb[TCA_GACT_PARMS - 1] == NULL) {
- printk("BUG: tcf_gact_init called with NULL params\n");
- return -1;
- }
+ return -EINVAL;
+ if (tb[TCA_GACT_PARMS - 1] == NULL)
+ return -EINVAL;

parm = RTA_DATA(tb[TCA_GACT_PARMS - 1]);
#ifdef CONFIG_GACT_PROB
@@ -99,7 +96,7 @@
if (p == NULL) {
p = tcf_hash_create(parm, est, a, sizeof(*p), ovr, bind); if (p == NULL)
- return -1;
+ return -ENOMEM;
else {
p->refcnt = 1;
ret = 1;