mbox series

[net,0/2] net/tls: fix W=1 build warnings

Message ID 20190509231407.25685-1-jakub.kicinski@netronome.com
Headers show
Series net/tls: fix W=1 build warnings | expand

Message

Jakub Kicinski May 9, 2019, 11:14 p.m. UTC 
Hi!

This small series cleans up two outstanding W=1 build
warnings in tls code.  Both are set but not used variables.
The first case looks fairly straightforward.  In the second
I think it's better to propagate the error code, even if
not doing some does not lead to a crash with current code.

Jakub Kicinski (2):
  net/tls: remove set but not used variables
  net/tls: handle errors from padding_length()

 net/tls/tls_device.c |  5 +----
 net/tls/tls_sw.c     | 30 ++++++++++++++++++++++--------
 2 files changed, 23 insertions(+), 12 deletions(-)

Comments

David Miller May 9, 2019, 11:37 p.m. UTC | #1 
From: Jakub Kicinski <jakub.kicinski@netronome.com>
Date: Thu,  9 May 2019 16:14:05 -0700

> This small series cleans up two outstanding W=1 build
> warnings in tls code.  Both are set but not used variables.
> The first case looks fairly straightforward.  In the second
> I think it's better to propagate the error code, even if
> not doing some does not lead to a crash with current code.

Series applied, thanks Jakub.