CID 1104354: The high byte of c is always 0, so don't bother shifting.

CID 1020958: Fix double unlock

CID 720453: Don't cast, so we restrict the range of values to the minimum and maximum indecies of the array

Hopefully coverity is smart enough to figure that out... Otherwise I guess we're checking the that uint8_t can't be bigger than UINT8_MAX :/

check return code

CID #1400026

we mean to ignore the return value

CID #1400023

check return code of read()

CID #1400021

ignore return code.

CID #1400018

we ignore the return code

CID #1400017

call request_data_get() to remove arguments to trigger function.

CID #1358901

We now have better checks for bad SSL linking.

CID #1358897

suppress output if there's no log.output

CID #1358896

check username, too.

CID #1358893

remove unnecessary check.

CID #1358883

eap_tls_data was already checked.

CID #1339404

Lie about **

Cast to quiet static analysis

CID 1409876: False positive

CID 1409875: Check data first before deref

CID 1409874: Always call va_end

CID 1409873: Don’t close the FD twice

CID 1409872: Always call va_end

CID 1409871: Check request_setup was successful

CID 1409869: Check fr_pair_alloc

CID 1409868: Check return of cf_data_find

CID 1409867: Missed break

 CID 1409866: Missed break

CID 1409865: Remove dead code

CID 1409864: Use the bool to check for multi

CID 1409863:  our_err is no longer used

CID 1409862: Missed check for NULL original packet

CID 140986: Check return code of cf_pair_parse

CID 1409860: Can’t use bounds checks on 64 bit integers

CID 1409859: Check error code of fr_radius_packet_send

CID 1409858: Fix typo

More formatting

Dynamically allocate modcall stack state

Formatting ’n inlining

Use normal return codes

Fix eap_peap_verify check

which wasn't actually incorrect, but rightly tripped PVS studio.  Seems the published draft doesn't actually match what we get in the tunnel.

Rename macros so it's clear which ones refer to EAP codes

memset_s might not be available

and yet more -V512 false positives

...and more V512 false positives

Ignore false positives in EAP sim code

We should probably just disable -V512, it's really stupid.

Fix scope, and use secure erase in mppe_keys

We should probably have a secret flag for attributes to ensure they get secure erased...

Unsetting indexes by calling SSL_set_ex_data with NULL is OK

Use correct buffer lengths in rlm_cram

Fix potential buffer overrun?

Ignore false positives in tacacs

Ignore stupid false positive in proto_detail.c

s/el/event_list/

Ignore false positives in udpfromto.c

PVS studio won't know about talloc_array_length... and it's more efficient too

Underflow is ok

Compare in_len to size of packet structure not void *

Ensure t->username is set

Move assignments out of conditions

Check packet allocation immediately

Ignore buffer underflow

shhhh pvs studio

Remove double checking of condition

Loop condition is useless

Remove erroneous assert in fr_request_to_state

Rename globals in radclient.c

Rename globals in process.c

Hmm this check just seems kinda broken

Formatting, and false positives

You're an odd type cast

Probably indeed was mistake

Only call things on session->ssl if it's not null

App data never used anywhere, and was NULL anyway‽

PVS studio is stupid

Fix comparators

cond_eval not used yet

Reduce scope of p so it can't be used after buffer goes out of scope

Assert first

Check we have a reply before checking its code

Check for Null fr_log_fp

Fix issues in the test utilities

Use isdigit instead of strtoul if we're just discarding the result

Fix order of assert

Month check should use a different iterator

We do actually want to hash the pointer, not the data

Can't goto finish if we're raising a signal

Use correct format for 64bit integers in worker.c

When we're localizing a header, copy the number of bytes in the header, not the number of bytes in a void *

Fix PVS studio issues in channel.c

Use correct format for 64bit integers in time.c

Use correct format for 64bit integers in atomic_queue.c

Fix pvs studio pedantry in jlibtool.c

Allow rlm_mruby to build, just remove its dependencies in travis

Because travis is still one major LTS version behind current *sigh*.

Alloc a pooled object for the stack, so we can use mutable frame extensions without leaking request pool memory

Compact bools in the stack frame into a bit field

FR_TLS_EX_IDENTITY is actualy a char **

Fix tls_validate_cert_cb to treat it as such.

Allow error and write callbacks to be passed into unlang_event_fd_add

Rename unlang_action to unlang_signal in some places

comment out unused code

shhh clang scan

Set data in all cases

Use common protocol encoder for RADIUS