diff options
author | Mark H Weaver <mhw@netris.org> | 2013-08-16 23:05:34 -0400 |
---|---|---|
committer | Mark H Weaver <mhw@netris.org> | 2013-08-16 23:05:34 -0400 |
commit | 88cff2ef1bd0d11987f034d2e7dec10c89f5719d (patch) | |
tree | ff0050987bd4fe30b3ac088e02a5ae583c0d3ccc | |
parent | 521c542199afa4f199746d5bbffc18a988cb30bc (diff) | |
parent | 088cfb7d761b01a2620d78f10e8dbcaa07485a32 (diff) | |
download | guile-88cff2ef1bd0d11987f034d2e7dec10c89f5719d.tar.gz |
Merge remote-tracking branch 'origin/stable-2.0'
-rw-r--r-- | libguile/bytevectors.c | 8 | ||||
-rw-r--r-- | test-suite/tests/bytevectors.test | 10 |
2 files changed, 12 insertions, 6 deletions
diff --git a/libguile/bytevectors.c b/libguile/bytevectors.c index 9093f49c3..56c00cb6e 100644 --- a/libguile/bytevectors.c +++ b/libguile/bytevectors.c @@ -1116,20 +1116,18 @@ SCM_DEFINE (scm_bytevector_sint_set_x, "bytevector-sint-set!", 5, 0, 0, \ SCM_VALIDATE_BYTEVECTOR (1, bv); \ SCM_VALIDATE_SYMBOL (2, endianness); \ - c_size = scm_to_uint (size); \ + c_size = scm_to_unsigned_integer (size, 1, (size_t) -1); \ \ c_len = SCM_BYTEVECTOR_LENGTH (bv); \ - if (SCM_UNLIKELY (c_len == 0)) \ + if (SCM_UNLIKELY (c_len < c_size)) \ lst = SCM_EOL; \ - else if (SCM_UNLIKELY (c_len < c_size)) \ - scm_out_of_range (FUNC_NAME, size); \ else \ { \ const char *c_bv; \ \ c_bv = (char *) SCM_BYTEVECTOR_CONTENTS (bv); \ \ - lst = scm_make_list (scm_from_uint (c_len / c_size), \ + lst = scm_make_list (scm_from_size_t (c_len / c_size), \ SCM_UNSPECIFIED); \ for (i = 0, pair = lst; \ i <= c_len - c_size; \ diff --git a/test-suite/tests/bytevectors.test b/test-suite/tests/bytevectors.test index 67fc6801f..524ce86b5 100644 --- a/test-suite/tests/bytevectors.test +++ b/test-suite/tests/bytevectors.test @@ -155,9 +155,17 @@ (let ((b (make-bytevector 0))) (null? (bytevector->uint-list b (endianness big) 2)))) + (pass-if "bytevector->sint-list [length < word size]" + (let ((b (make-bytevector 1))) + (null? (bytevector->sint-list b (endianness big) 2)))) + (pass-if-exception "bytevector->sint-list [out-of-range]" exception:out-of-range - (bytevector->sint-list (make-bytevector 6) (endianness little) 8)) + (bytevector->sint-list (make-bytevector 6) (endianness little) -1)) + + (pass-if-exception "bytevector->uint-list [out-of-range]" + exception:out-of-range + (bytevector->uint-list (make-bytevector 6) (endianness little) 0)) (pass-if "bytevector->sint-list [off-by-one]" (equal? (bytevector->sint-list (make-bytevector 31 #xff) |