net: sched: cls_flower: Fix an error code in fl_tmplt_create()
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 3 Aug 2018 19:27:55 +0000 (22:27 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 6 Aug 2018 00:25:46 +0000 (17:25 -0700)
We forgot to set the error code on this path, so we return NULL instead
of an error pointer.  In the current code kzalloc() won't fail for small
allocations so this doesn't really affect runtime.

Fixes: b95ec7eb3b4d ("net: sched: cls_flower: implement chain templates")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/cls_flower.c

index e8bd08ba998a9bf6c12cb13e32bd297fc7cf10cd..a3b69bb6f4b0f30a0612429c08cf765edd1162c3 100644 (file)
@@ -1250,8 +1250,10 @@ static void *fl_tmplt_create(struct net *net, struct tcf_chain *chain,
                goto errout_tb;
 
        tmplt = kzalloc(sizeof(*tmplt), GFP_KERNEL);
-       if (!tmplt)
+       if (!tmplt) {
+               err = -ENOMEM;
                goto errout_tb;
+       }
        tmplt->chain = chain;
        err = fl_set_key(net, tb, &tmplt->dummy_key, &tmplt->mask, extack);
        if (err)