diff options
| author | Samuel Cabrero <scabrero@suse.de> | 2019-02-18 21:06:02 +0100 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2019-07-22 16:49:16 +0000 |
| commit | dde08e8f262633c017c8288991c9e894cb72d0d8 (patch) | |
| tree | 07b20588db0b5ec16fb46029cc569ad31fad89b6 /testsuite | |
| parent | f7529fffce6de70a6914e17f91876d721e02b219 (diff) | |
| download | samba-dde08e8f262633c017c8288991c9e894cb72d0d8.tar.gz | |
s3:rpc_server: Return NTSTATUS in is_known_pipename function
Signed-off-by: Samuel Cabrero <scabrero@suse.de>
Reviewed-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'testsuite')
| -rw-r--r-- | testsuite/unittests/test_sambafs_srv_pipe.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/testsuite/unittests/test_sambafs_srv_pipe.c b/testsuite/unittests/test_sambafs_srv_pipe.c index 553f5309928..161f7131e8e 100644 --- a/testsuite/unittests/test_sambafs_srv_pipe.c +++ b/testsuite/unittests/test_sambafs_srv_pipe.c @@ -38,26 +38,26 @@ static int teardown_samr(void **state) static void test_is_known_pipename(void **state) { struct ndr_syntax_id syntax_id = ndr_table_samr.syntax_id; - bool is_pipename_ok; + NTSTATUS status; - is_pipename_ok = is_known_pipename("samr", &syntax_id); - assert_true(is_pipename_ok); + status = is_known_pipename("samr", &syntax_id); + assert_true(NT_STATUS_IS_OK(status)); } static void test_is_known_pipename_slash(void **state) { struct ndr_syntax_id syntax_id = ndr_table_samr.syntax_id; - bool is_pipename_ok; char dummy_module_path[4096] = {0}; const char *module_env; + NTSTATUS status; snprintf(dummy_module_path, sizeof(dummy_module_path), "%s/bin/modules/rpc/test_dummy_module.so", SRCDIR); - is_pipename_ok = is_known_pipename(dummy_module_path, &syntax_id); - assert_false(is_pipename_ok); + status = is_known_pipename(dummy_module_path, &syntax_id); + assert_true(NT_STATUS_IS_ERR(status)); module_env = getenv("UNITTEST_DUMMY_MODULE_LOADED"); assert_null(module_env); |