From: Andreas Schneider Date: Tue, 30 Apr 2024 15:31:00 +0000 (+0200) Subject: s4:torture: Add missing NULL checks in spoolss.c X-Git-Tag: tdb-1.4.11~358 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=4a48148a7d830fb0ec6884f39b2ab1e5d41f2181;p=thirdparty%2Fsamba.git s4:torture: Add missing NULL checks in spoolss.c source4/torture/rpc/spoolss.c: In function ‘check_printer_driver_file.isra’: source4/torture/rpc/spoolss.c:10850:58: error: ‘%s’ directive argument is null [-Werror=format-overflow=] 10850 | const char *remote_name = talloc_asprintf(tctx, "%s\\%d\\%s", | ^~ Signed-off-by: Andreas Schneider Reviewed-by: Stefan Metzmacher --- diff --git a/source4/torture/rpc/spoolss.c b/source4/torture/rpc/spoolss.c index 05a0aef7e43..867b94ba654 100644 --- a/source4/torture/rpc/spoolss.c +++ b/source4/torture/rpc/spoolss.c @@ -10846,15 +10846,22 @@ static bool check_printer_driver_file(struct torture_context *tctx, struct torture_driver_context *d, const char *file_name) { - const char *remote_arch_dir = driver_directory_dir(d->remote.driver_directory); - const char *remote_name = talloc_asprintf(tctx, "%s\\%d\\%s", - remote_arch_dir, - d->info8.version, - file_name); + const char *remote_arch_dir = NULL; + const char *remote_name = NULL; int fnum; torture_assert(tctx, (file_name && strlen(file_name) != 0), "invalid filename"); + remote_arch_dir = driver_directory_dir(d->remote.driver_directory); + torture_assert_not_null(tctx, remote_arch_dir, "remote_arch_dir is null"); + + remote_name = talloc_asprintf(tctx, + "%s\\%d\\%s", + remote_arch_dir, + d->info8.version, + file_name); + torture_assert_not_null(tctx, remote_name, "renote_name is null"); + torture_comment(tctx, "checking for driver file at %s\n", remote_name); fnum = smbcli_open(cli->tree, remote_name, O_RDONLY, DENY_NONE);