diff options
| author | Andreas Schneider <asn@samba.org> | 2023-04-20 13:09:26 +0200 |
|---|---|---|
| committer | Andreas Schneider <asn@cryptomilk.org> | 2023-04-27 07:21:33 +0000 |
| commit | 0c6fb4bfb01254d565d5b8308b24447d926b5085 (patch) | |
| tree | a03785a6fd1e34fde331a03142ac036b774a9943 /dfs_server | |
| parent | 3e6a6c00cc09fedefa5c02f24d7852fea9a4c484 (diff) | |
| download | samba-0c6fb4bfb01254d565d5b8308b24447d926b5085.tar.gz | |
dfs_server: Fix debug statement if searched_site is NULL
In file included from source4/include/includes.h:61,
from dfs_server/dfs_server_ad.c:21:
dfs_server/dfs_server_ad.c: In function ‘get_dcs.constprop’:
lib/util/debug.h:200:12: error: ‘%s’ directive argument is null [-Werror=format-overflow=]
200 | && (dbgtext body) )
| ~^~~~~~~~~~~~~
dfs_server/dfs_server_ad.c:462:25: note: in expansion of macro ‘DEBUG’
462 | DEBUG(2,(__location__ ": Site: %s %s\n",
| ^~~~~
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'dfs_server')
| -rw-r--r-- | dfs_server/dfs_server_ad.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/dfs_server/dfs_server_ad.c b/dfs_server/dfs_server_ad.c index b97d0882f94..f992042966e 100644 --- a/dfs_server/dfs_server_ad.c +++ b/dfs_server/dfs_server_ad.c @@ -459,8 +459,11 @@ static NTSTATUS get_dcs(TALLOC_CTX *ctx, struct ldb_context *ldb, if (searched_site == NULL || strcmp(searched_site, site_name) != 0) { - DEBUG(2,(__location__ ": Site: %s %s\n", - searched_site, site_name)); + DEBUG(2, + (__location__ ": Site: %s %s\n", + searched_site != NULL ? searched_site + : "UNKNOWN", + site_name)); /* * Do all the site but the one of the client |