diff options
| author | Lorenzo Colitti <lorenzo@google.com> | 2013-07-04 00:12:40 +0900 |
|---|---|---|
| committer | Bitan Biswas <bbiswas@nvidia.com> | 2013-12-03 23:04:47 -0800 |
| commit | 861112e937684f20307545083b23a2ad76535330 (patch) | |
| tree | 610163992232c047ed4de5b40d636f2105d2c076 /net/ipv6 | |
| parent | 90d045f56779fd1feb322d61fd4d7821ce907885 (diff) | |
net: ipv6: fix wrong ping_v6_sendmsg return value
ping_v6_sendmsg currently returns 0 on success. It should return
the number of bytes written instead.
Bug: 9469865
Bug 1412141
Change-Id: I14aad6018f0c3fd92dad315bc3ade04c0b3982e0
Signed-off-by: Lorenzo Colitti <lorenzo@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
(cherry picked from commit 5f0fd6105f491b8b435863e98f7e96d617e3da84)
Reviewed-on: http://git-master/r/336272
GVS: Gerrit_Virtual_Submit
Reviewed-by: Om Prakash Singh <omp@nvidia.com>
Tested-by: Om Prakash Singh <omp@nvidia.com>
Reviewed-by: Narayan Reddy <narayanr@nvidia.com>
Reviewed-by: Bitan Biswas <bbiswas@nvidia.com>
Reviewed-on: http://git-master/r/337767
Diffstat (limited to 'net/ipv6')
| -rw-r--r-- | net/ipv6/ping.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/ipv6/ping.c b/net/ipv6/ping.c index 9ca1f1ac7a14..858788da5a2f 100644 --- a/net/ipv6/ping.c +++ b/net/ipv6/ping.c @@ -214,5 +214,8 @@ int ping_v6_sendmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg, } release_sock(sk); - return err; + if (err) + return err; + + return len; } |