diff options
| author | Sam Thursfield <sam@afuera.me.uk> | 2016-09-03 11:05:25 +0100 |
|---|---|---|
| committer | Sam Thursfield <sam@afuera.me.uk> | 2016-09-03 11:28:19 +0100 |
| commit | 9b419faeeced8e2c4fc9d5c6dc761268fbe70578 (patch) | |
| tree | be4c9856e0b851f77bc81e739c667fe949e42cca /tests/libtracker-sparql | |
| parent | 2641562e5bd3d16dd3d67a5bd3d06e3006ec345b (diff) | |
| download | tracker-9b419faeeced8e2c4fc9d5c6dc761268fbe70578.tar.gz | |
tests: Avoid horrible hang in test-tracker-resource
Seems that missing the 'return' statement from a
g_test_trap_subprocess() block can result in hanging Linux completely;
some kind of fork bomb I guess. The problem only hits when the test is
failing, so I didn't spot the mistake before.
Diffstat (limited to 'tests/libtracker-sparql')
| -rw-r--r-- | tests/libtracker-sparql/tracker-resource-test.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/libtracker-sparql/tracker-resource-test.c b/tests/libtracker-sparql/tracker-resource-test.c index cab3524a5..4c86fe865 100644 --- a/tests/libtracker-sparql/tracker-resource-test.c +++ b/tests/libtracker-sparql/tracker-resource-test.c @@ -134,8 +134,10 @@ test_resource_get_set_pointer_validation (void) resource = tracker_resource_new ("http://example.com/resource"); - /* This will trigger a g_warning(), and abort. */ + /* This should trigger a g_warning(), and abort. */ tracker_resource_set_string (resource, "http://example.com/1", NULL); + + return; } g_test_trap_subprocess (NULL, 0, 0); |