diff options
author | Marek Chalupa <mchqwerty@gmail.com> | 2014-04-11 11:48:55 +0200 |
---|---|---|
committer | Kristian Høgsberg <krh@bitplanet.net> | 2014-04-16 22:31:44 -0700 |
commit | 06f2fa1a609f5c263c7b8ea3cb8a967f9ecde338 (patch) | |
tree | 79c4d5a2441098a709089196c1a2ac1869a7853e /tests/bad-buffer-test.c | |
parent | 9e9512f017e033c6a1719c0647cc73ec228c715d (diff) | |
download | weston-06f2fa1a609f5c263c7b8ea3cb8a967f9ecde338.tar.gz |
tests: fix bad-buffer-test
bad-buffer-test is FAIL_TEST and every assert() (or even SIGSEGV signal)
make it pass. It shouldn't be so for example when assert() is invoked
when a client couldn't connect to display.
Make sure that only relevant asserts make the test pass
and the other make it fail (by returning 0)
Diffstat (limited to 'tests/bad-buffer-test.c')
-rw-r--r-- | tests/bad-buffer-test.c | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/tests/bad-buffer-test.c b/tests/bad-buffer-test.c index 6eae313e..86e0299b 100644 --- a/tests/bad-buffer-test.c +++ b/tests/bad-buffer-test.c @@ -25,6 +25,8 @@ #include <unistd.h> #include <sys/types.h> +#include <stdio.h> +#include <signal.h> #include "../shared/os-compatibility.h" #include "weston-test-client-helper.h" @@ -58,12 +60,34 @@ create_bad_shm_buffer(struct client *client, int width, int height) return buffer; } +static void sighandler(int signum) +{ + /* this means failure */ + exit(0); +} + FAIL_TEST(test_truncated_shm_file) { struct client *client; struct wl_buffer *bad_buffer; struct wl_surface *surface; int frame; + struct sigaction new_action, old_action; + + /* until the bad buffer creation, the SIGABRT or SIGSEGV signals + * should fail the test. That means returning 0 */ + new_action.sa_handler = sighandler; + sigemptyset(&new_action.sa_mask); + new_action.sa_flags = 0; + + if (sigaction(SIGSEGV, &new_action, NULL) != 0) { + fprintf(stderr, "Failed setting new sigaction for SIGSEGV"); + exit(0); + } + if (sigaction(SIGABRT, &new_action, &old_action) != 0) { + fprintf(stderr, "Failed setting new sigaction for SIGABRT"); + exit(0); + } client = client_create(46, 76, 111, 134); assert(client); @@ -71,6 +95,12 @@ FAIL_TEST(test_truncated_shm_file) bad_buffer = create_bad_shm_buffer(client, 200, 200); + /* from this point we expect the signal */ + if (sigaction(SIGABRT, &old_action, NULL) != 0) { + fprintf(stderr, "Failed setting old sigaction for SIGABRT"); + exit(0); + } + wl_surface_attach(surface, bad_buffer, 0, 0); wl_surface_damage(surface, 0, 0, 200, 200); frame_callback_set(surface, &frame); |