diff options
| author | stephen hemminger <stephen@networkplumber.org> | 2013-09-26 17:40:11 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-09-30 15:47:43 -0400 |
| commit | 0c4e4020f0145ed84b0ef9e3fb79c4a022e77e47 (patch) | |
| tree | 04694b8d25bba52dd554843465035a0e48261b08 | |
| parent | 90de527d7ec77e509a22596d8f9eae9b90fd28ca (diff) | |
| download | op-kernel-dev-0c4e4020f0145ed84b0ef9e3fb79c4a022e77e47.zip op-kernel-dev-0c4e4020f0145ed84b0ef9e3fb79c4a022e77e47.tar.gz | |
qdisc: meta return ENOMEM on alloc failure
Rather than returning earlier value (EINVAL), return ENOMEM if
kzalloc fails. Found while reviewing to find another EINVAL condition.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/sched/em_meta.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/sched/em_meta.c b/net/sched/em_meta.c index 7c3de6f..e5cef956 100644 --- a/net/sched/em_meta.c +++ b/net/sched/em_meta.c @@ -793,8 +793,10 @@ static int em_meta_change(struct tcf_proto *tp, void *data, int len, goto errout; meta = kzalloc(sizeof(*meta), GFP_KERNEL); - if (meta == NULL) + if (meta == NULL) { + err = -ENOMEM; goto errout; + } memcpy(&meta->lvalue.hdr, &hdr->left, sizeof(hdr->left)); memcpy(&meta->rvalue.hdr, &hdr->right, sizeof(hdr->right)); |
