aboutsummaryrefslogtreecommitdiffstats
path: root/src/tls
diff options
context:
space:
mode:
authorJouni Malinen <jouni@qca.qualcomm.com>2015-12-28 16:15:18 (GMT)
committerJouni Malinen <j@w1.fi>2015-12-28 16:15:18 (GMT)
commitc6231b5e1ead720f8c597686260383fc9c351c68 (patch)
tree1dae5e467c58b1703cd2b6f7b5a2e76ad950e470 /src/tls
parent15a68138c1da0ec509d09bbd143c17abc720b29a (diff)
downloadhostap-c6231b5e1ead720f8c597686260383fc9c351c68.zip
hostap-c6231b5e1ead720f8c597686260383fc9c351c68.tar.gz
hostap-c6231b5e1ead720f8c597686260383fc9c351c68.tar.bz2
TLS: Remove storing of never-read value
While this could in theory be claimed to be ready for something to be added to read a field following the server_write_IV, it does not look likely that such a use case would show up. As such, just remove the unused incrementing of pos at the end of the function to get rid of a useless static analyzer complaint. Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
Diffstat (limited to 'src/tls')
-rw-r--r--src/tls/tlsv1_client.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/src/tls/tlsv1_client.c b/src/tls/tlsv1_client.c
index cc404c1..9bc0d21 100644
--- a/src/tls/tlsv1_client.c
+++ b/src/tls/tlsv1_client.c
@@ -111,7 +111,6 @@ int tls_derive_keys(struct tlsv1_client *conn,
pos += conn->rl.iv_size;
/* server_write_IV */
os_memcpy(conn->rl.read_iv, pos, conn->rl.iv_size);
- pos += conn->rl.iv_size;
} else {
/*
* Use IV field to set the mask value for TLS v1.1. A fixed