Andy Green
c21467a8f0
coverity: 50976: check fsync return for cert write
2020-08-19 07:23:19 +01:00
Andy Green
e2dcbdc81a
coverity: 49785: confirm initgroups worked
2020-08-19 07:21:55 +01:00
Andy Green
258ee1886f
coverity: 37468: confirm gethostname worked
2020-08-19 07:19:48 +01:00
Andy Green
d69b91d2d9
coverity: 62123: explicitly check things to guide coverity
...
Coverity sees we sometimes check if header length is 0 and extrapolates
from that not checking header length return is a violation. But often
we are OK if the header length is 0 and there is no error return to
check from that.
It also doesn't understand that if we saw a nonzero length for a header,
then we are going to get a non-null simple_ptr() return for sure. Just
give up and explicitly, unneccessarily check everything so coverity can
stop telling us about it.
2020-08-19 07:11:59 +01:00
Andy Green
944ef24e67
coverity: 10600: make sure vhosts get off the pending destruction list if on it
2020-08-19 06:52:47 +01:00
Andy Green
a8315807e7
coverity: 10417: move goto inside preprocessor conditional that needs it
...
Otherwise coverity sees it with !defined(LWS_ROLE_WS) sitting there doing nothing
2020-08-19 06:46:19 +01:00
Andy Green
7ff64b3c42
coverity: 10117: help coverity understand simple_ptr cannot be NULL
2020-08-19 06:40:13 +01:00
Andy Green
a743a2b234
coverity: 62207: we only need to compute the flag if LWS_WITH_SYS_ASYNC_DNS
2020-08-18 14:00:54 +01:00
Andy Green
a6b4bd8bfc
coverity: 62374: no need to init n
2020-08-18 14:00:54 +01:00
Andy Green
9679222070
coverity: 62147: setting retcode var is cruft
2020-08-18 14:00:54 +01:00
Andy Green
76ee92afb4
coverity: 62574: wsi mux close: dumb down iterator so no dead code
2020-08-18 14:00:54 +01:00
Andy Green
cc5fb48598
coverity: 21034: dont bother assigning shutdown api return
2020-08-18 14:00:54 +01:00
Andy Green
ad6dfd3df6
coverity: 21071: no need to init accept_fd
2020-08-18 14:00:54 +01:00
Andy Green
7ded454d2d
coverity: 62570: back up assert with unconditional NULL check
2020-08-18 14:00:54 +01:00
Andy Green
84a8ada0fd
coverity: 62181: remove needless wsi check
2020-08-18 14:00:54 +01:00
Andy Green
d5497d5f55
coverity: 62535: another simple_ptr NULL check after confirmed by other means
2020-08-18 14:00:54 +01:00
Andy Green
520befb554
coverity: 61708: privkey required for mbedtls
2020-08-18 14:00:54 +01:00
Andy Green
7b847f8315
coverity: 62212: coverity says its dead code
2020-08-18 14:00:54 +01:00
Andy Green
264c98c2c9
coverity: 62494: check system blob supposed to be used in h1 at runtime
2020-08-18 14:00:54 +01:00
Andy Green
80ea883796
coverity: 62550: back up assert with runtime NULL check
2020-08-18 14:00:54 +01:00
Andy Green
acf42bec9d
coverity: 62211: use lws_protocol_init retcode in context init
2020-08-18 14:00:54 +01:00
Andy Green
6b09e5f8f7
coverity: 62313: static blob cant be null but coverity needs convincing
2020-08-18 14:00:54 +01:00
Andy Green
398a9e1bad
coverity: 62134: protect debug-only statement with preprocessor conditional
...
Compiler has no problem with it but coverity complains nobody uses p in release build
2020-08-18 14:00:54 +01:00
Andy Green
598a82ca1d
coverity: 62512: remove unused assignment
2020-08-18 14:00:54 +01:00
Andy Green
5ef6c548d8
coverity: 62488 62253: comment NOP left in for extensibility
2020-08-18 14:00:54 +01:00
Andy Green
260bd38d56
coverity: 62319: coverity thinks we must always check blob size
2020-08-18 14:00:54 +01:00
Andy Green
d3d8b3a74c
coverity: 62370: reassure coverity static blob always exists
2020-08-18 14:00:54 +01:00
Andy Green
03ab5b5247
coverity: 62376: reassure coverity static blob always exists
2020-08-18 14:00:54 +01:00
Andy Green
80395f479f
coverity: 62431: reassure coverity static blob always exists
2020-08-18 14:00:54 +01:00
Andy Green
fc9a23abeb
coverity: 62581: lwa auth: handle illegal blob index
2020-08-18 14:00:54 +01:00
Andy Green
1892af1ce3
coverity: 40529: add needless check on NULL simple_ptr after confirmed it has nonzero length
2020-08-18 14:00:54 +01:00
Andy Green
630391e0fb
coverity: 50320: show coverity protocol can always be found by name
2020-08-18 14:00:54 +01:00
Andy Green
b1281f1f3a
coverity: 50773: no need to set m any more when transitioning and exiting
2020-08-18 14:00:54 +01:00
Andy Green
cdbf86fe4a
coverity: 51248: convince coverity we have an ah attached before dereference during parse
2020-08-18 14:00:54 +01:00
Andy Green
b63c7f1e8e
coverity: 62131: check when logging role name for NULL
2020-08-18 14:00:54 +01:00
Andy Green
06005d14b4
coverity: 62154: calculation needed if WITH_FILE_OPS
2020-08-18 09:34:50 +01:00
Andy Green
048604751c
coverity: 62584: init m only inside preprocessor conditional that needs it
2020-08-18 09:28:40 +01:00
Andy Green
60e2c65208
coverity: 62333; no need to init cce as set on all paths that use it
2020-08-18 09:19:34 +01:00
Andy Green
400355fdc3
coverity: 62458: coverity doesnt understand nonzero header length means simple_ptr cannot be NULL
2020-08-18 09:19:33 +01:00
Andy Green
9745c5cca8
coverity: 62477: explicitly check wsi->protocol even though client creation sets it
2020-08-18 09:11:45 +01:00
Andy Green
192b76c89b
coverity: 62540: explicitly check for NULL even though cannot be
2020-08-18 09:08:03 +01:00
Andy Green
0f173e0bbe
coverity: 62622: remove unused line
2020-08-18 09:00:39 +01:00
Andy Green
52182da689
sspc: protect wsi from NULL not cwsi
2020-08-17 16:27:05 +01:00
Andy Green
83912f40e8
sspc: proxy: extend DESTROY_ME
2020-08-17 14:36:33 +01:00
Andy Green
c6e1352e99
sspc: add translation helpers and comments about pss usage between client and proxy
...
Helpers remove casts and derefs.
Add additional pointer arithmetic in client_pss_to_sspc_h() helper to
remove dependency on handle_offset being the first thing in the userdata
Make the helper names explicit for different proxy and client pss handling,
so it should be clearer that client helpers belong in a client section and
vice versa.
2020-08-17 14:32:23 +01:00
Jed Lu
80992b0ebd
sspc: client: audit all deserialize *pss references and confirmed checked for NULL
2020-08-17 06:57:30 +01:00
Andy Green
e8cbfea22d
lws_smd: take care about build with disabled
2020-08-16 05:35:56 +01:00
Andy Green
7692d920a8
coverity: 244441: strerror broken on some platforms
...
Some platforms have strerror but it's broken according to Coverity.
Let's avoid it
2020-08-14 09:14:24 +01:00
Andy Green
491f0f6068
coverity: 324943: confirm get_random worked
...
In normal systems this doesn't have a way to fail. But check it for
consistency.
2020-08-14 09:07:27 +01:00
Andy Green
aec2bdec2f
coverity: 324897: mqtt: check unsub var for NULL
2020-08-14 09:02:29 +01:00