diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2018-01-02 10:02:19 +0000 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2018-01-02 10:02:19 +0000 |
| commit | 7888da95832d50a87bbfdb9f40620ddc66f94b3c (patch) | |
| tree | c66fcc953405353255c20de8426863fd1c7406db /fs/afs | |
| parent | 98801506552593c9b8ac11021b0cdad12cab4f6b (diff) | |
| download | linux-7888da95832d50a87bbfdb9f40620ddc66f94b3c.tar.gz linux-7888da95832d50a87bbfdb9f40620ddc66f94b3c.tar.bz2 linux-7888da95832d50a87bbfdb9f40620ddc66f94b3c.zip | |
afs: Potential uninitialized variable in afs_extract_data()
Smatch warns that:
fs/afs/rxrpc.c:922 afs_extract_data()
error: uninitialized symbol 'remote_abort'.
Smatch is right that "remote_abort" might be uninitialized when we pass
it to afs_set_call_complete(). I don't know if that function uses the
uninitialized variable. Anyway, the comment for rxrpc_kernel_recv_data(),
says that "*_abort should also be initialised to 0." and this patch does
that.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'fs/afs')
| -rw-r--r-- | fs/afs/rxrpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/afs/rxrpc.c b/fs/afs/rxrpc.c index ea1460b9b71a..e1126659f043 100644 --- a/fs/afs/rxrpc.c +++ b/fs/afs/rxrpc.c @@ -885,7 +885,7 @@ int afs_extract_data(struct afs_call *call, void *buf, size_t count, { struct afs_net *net = call->net; enum afs_call_state state; - u32 remote_abort; + u32 remote_abort = 0; int ret; _enter("{%s,%zu},,%zu,%d", |