bartoszek/zephyr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

net: tls: Use EINVAL for NULL pointer error in option setters

Browse Source 
Originally EFAULT was used to indicate NULL pointer error in TLS option
set/get functions. EINVAL was suggested to be more apropriate error code
for this case, hence replace it.

Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
This commit is contained in:
Robert Lubos 2018-08-13 10:44:56 +02:00 committed by Jukka Rissanen
parent b7c254fc40
commit 7c487ba04a
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
subsys/net/lib/sockets/sockets_tls.c
View File

@ -888,7 +888,7 @@ static int tls_opt_sec_tag_list_set(struct net_context *context,
int sec_tag_cnt;
if (!optval) {
return -EFAULT;
return -EINVAL;
}
if (optlen % sizeof(sec_tag_t) != 0) {
@ -945,7 +945,7 @@ static int tls_opt_ciphersuite_list_set(struct net_context *context,
int cipher_cnt;
if (!optval) {
return -EFAULT;
return -EINVAL;
}
if (optlen % sizeof(int) != 0) {
@ -1022,7 +1022,7 @@ static int tls_opt_peer_verify_set(struct net_context *context,
int *peer_verify;
if (!optval) {
return -EFAULT;
return -EINVAL;
}
if (optlen != sizeof(int)) {
@ -1048,7 +1048,7 @@ static int tls_opt_role_set(struct net_context *context, const void *optval,
int *role;
if (!optval) {
return -EFAULT;
return -EINVAL;
}
if (optlen != sizeof(int)) {
@ -1720,7 +1720,7 @@ int ztls_getsockopt(int sock, int level, int optname,
}
if (!optval || !optlen) {
return -EFAULT;
return -EINVAL;
}
switch (optname) {