mirror/guile
mirror/guile
1
Fork 0
mirror of https://git.savannah.gnu.org/git/guile.git synced 2025-05-20 03:30:27 +02:00
Code Activity

'load-thunk-from-memory' reports the correct error.

Browse source 
Previously 'load-thunk-from-memory' would often throw to 'system-error'
when passed an incorrect ELF file, leading to incorrect error messages.

* libguile/loader.c (load_thunk_from_memory): Reset 'errno' when
'check_elf_header' returns non-NULL.
* test-suite/tests/vm.test: New file.
* test-suite/Makefile.am (SCM_TESTS): Add it.
This commit is contained in:
Ludovic Courtès 2017-12-22 11:40:27 +01:00
parent 6d391bf1a4
commit deef64a739
3 changed files with 60 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
libguile/loader.c
View file

@ -1,5 +1,5 @@
/* Copyright (C) 2001, 2009, 2010, 2011, 2012
* 2013, 2014, 2015 Free Software Foundation, Inc.
* 2013, 2014, 2015, 2017 Free Software Foundation, Inc.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public License
@ -362,7 +362,10 @@ load_thunk_from_memory (char *data, size_t len, int is_read_only)
header = (Elf_Ehdr*) data;
if ((err_msg = check_elf_header (header)))
{
errno = 0; /* not an OS error */
goto cleanup;
}
if (header->e_phnum == 0)
ABORT ("no loadable segments");

1
test-suite/Makefile.am
View file

@ -189,6 +189,7 @@ SCM_TESTS = tests/00-initial-env.test \
tests/version.test \
tests/vectors.test \
tests/vlist.test \
tests/vm.test \
tests/weaks.test \
tests/web-client.test \
tests/web-http.test \

54
test-suite/tests/vm.test Normal file
View file

@ -0,0 +1,54 @@
;;;; vm.test --- tests for the ELF machinery and VM -*- scheme -*-
;;;; Copyright (C) 2017 Free Software Foundation, Inc.
;;;;
;;;; This library is free software; you can redistribute it and/or
;;;; modify it under the terms of the GNU Lesser General Public
;;;; License as published by the Free Software Foundation; either
;;;; version 3 of the License, or (at your option) any later version.
;;;;
;;;; This library is distributed in the hope that it will be useful,
;;;; but WITHOUT ANY WARRANTY; without even the implied warranty of
;;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
;;;; Lesser General Public License for more details.
;;;;
;;;; You should have received a copy of the GNU Lesser General Public
;;;; License along with this library; if not, write to the Free Software
;;;; Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
(define-module (tests vm)
#:use-module (test-suite lib)
#:use-module (system vm loader)
#:use-module (system vm elf)
#:use-module (rnrs bytevectors))
(define (elf->bytevector elf)
(let ((bv (make-bytevector 1000)))
(write-elf-header bv elf)
bv))
(with-test-prefix "load-thunk-from-memory"
(pass-if-exception "wrong byte order"
'(misc-error . "does not have native byte order")
;; This used to throw to 'system-error' with whatever value errno had.
(begin
(false-if-exception (open-output-file "/does-not-exist"))
(load-thunk-from-memory
(elf->bytevector
(make-elf #:byte-order (if (eq? (native-endianness)
(endianness little))
(endianness big)
(endianness
little))
#:shoff 0)))))
(pass-if-exception "wrong OS ABI"
'(misc-error . "OS ABI")
;; This used to throw to 'system-error' with whatever value errno had.
(begin
(false-if-exception (open-output-file "/does-not-exist"))
(load-thunk-from-memory
(elf->bytevector
(make-elf #:abi ELFOSABI_TRU64 ;RIP
#:shoff 0))))))