summaryrefslogtreecommitdiff
path: root/libguile/intrinsics.h
diff options
context:
space:
mode:
authorMark H Weaver <mhw@netris.org>2019-04-01 22:11:35 -0400
committerAndy Wingo <wingo@pobox.com>2019-05-23 17:40:08 +0200
commit980d8265c2dc35d6e02e540c9041cbf0975dfede (patch)
tree067f18effb6989e30fa872310ff3bddd920a8ac8 /libguile/intrinsics.h
parentb38d9a1527a9d796034bf721c9a266d261db5ad0 (diff)
downloadguile-980d8265c2dc35d6e02e540c9041cbf0975dfede.tar.gz
Avoid passing NULL to 'memcpy' and 'memcmp'.
Reported by Jeffrey Walton <noloader@gmail.com> in <https://lists.gnu.org/archive/html/guile-devel/2019-03/msg00001.html>. Note that C11 section 7.1.4 (Use of library functions) states that: "unless explicitly stated otherwise in the detailed descriptions [of library functions] that follow: If an argument to a function has an invalid value (such as ... a null pointer ...) ..., the behavior is undefined." Note that 'strxfrm' is an example of a standard C function that explicitly states otherwise, allowing NULL to be passed in the first argument if the size argument is zero, but no similar allowance is specified for 'memcpy' or 'memcmp'. * libguile/bytevectors.c (scm_uniform_array_to_bytevector): Call memcpy only if 'byte_len' is non-zero. * libguile/srfi-14.c (charsets_equal): Call memcmp only if the number of ranges is non-zero. * libguile/stime.c (setzone): Pass 1-character buffer to 'scm_to_locale_stringbuf', instead of NULL. * libguile/strings.c (scm_to_locale_stringbuf): Call memcpy only if the number of bytes to copy is non-zero.
Diffstat (limited to 'libguile/intrinsics.h')
0 files changed, 0 insertions, 0 deletions