diff options
| author | Vakul Garg <vakul.garg@nxp.com> | 2018-09-24 13:39:49 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-09-24 22:24:52 +0300 |
| commit | 4128c0cfb1d74cb1fa2a2fb6824348b349d30f04 (patch) | |
| tree | eff037f4f01dd804bbb808073658d184765d5b31 /net/tls | |
| parent | 9932a29ab1be1427a2ccbdf852a0f131f2849685 (diff) | |
| download | linux-4128c0cfb1d74cb1fa2a2fb6824348b349d30f04.tar.xz | |
tls: Fixed uninitialised vars warning
In tls_sw_sendmsg() and tls_sw_sendpage(), it is possible that the
uninitialised variable 'ret' gets passed to sk_stream_error(). So
initialise local variable 'ret' to '0. The warnings were detected by
'smatch' tool.
Fixes: a42055e8d2c3 ("net/tls: Add support for async encryption")
Signed-off-by: Vakul Garg <vakul.garg@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tls')
| -rw-r--r-- | net/tls/tls_sw.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/tls/tls_sw.c b/net/tls/tls_sw.c index d30d65bf0753..bf03f32aa983 100644 --- a/net/tls/tls_sw.c +++ b/net/tls/tls_sw.c @@ -690,7 +690,7 @@ int tls_sw_sendmsg(struct sock *sk, struct msghdr *msg, size_t size) int record_room; int num_zc = 0; int orig_size; - int ret; + int ret = 0; if (msg->msg_flags & ~(MSG_MORE | MSG_DONTWAIT | MSG_NOSIGNAL)) return -ENOTSUPP; @@ -882,8 +882,8 @@ int tls_sw_sendpage(struct sock *sk, struct page *page, int num_async = 0; bool full_record; int record_room; + int ret = 0; bool eor; - int ret; if (flags & ~(MSG_MORE | MSG_DONTWAIT | MSG_NOSIGNAL | MSG_SENDPAGE_NOTLAST)) |