[gnutls-devel] GnuTLS | Datum.c cleanup (!1002)
Development of GNU's TLS library
gnutls-devel at lists.gnutls.org
Tue May 28 16:49:18 CEST 2019
I've amended my previous patch, to avoid the `assert()`.
> `_gnutls_set_strdatum` also allows `data == NULL` as long as the `data_size` argument is 0.
I'll comment inline.
> `warn_unused_result` immediately warns us if we introduce an unchecked call in the future.
> `nonnull((1))` triggers an error on clang's build-scan if the first argument could be NULL.
My concern was a function calling `set()`, then acting assuming the value is initialized to either `NULL` or something else irrespective of failure or success.
> All calls to `_gnutls_set_datum` and `_gnutls_set_strdatum` now check their return value and fail out on error. I also manually skimmed through the places where we call one of those functions. LGTM.
Thank you for that.
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/merge_requests/1002#note_175390188
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Gnutls-devel