From 43b563fabd2d86161677fd2edd83ff1a9f1625e3 Mon Sep 17 00:00:00 2001 From: Matthias Andree Date: Thu, 29 Apr 2010 10:44:44 +0200 Subject: Fix two warning: the address of * will always evaluate as 'true'. --- NEWS | 1 + smbutil.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 96bcfd54..7a7823b5 100644 --- a/NEWS +++ b/NEWS @@ -88,6 +88,7 @@ fetchmail-6.3.17 (not yet released): are now helpful pointers to --sslcertpath and c_rehash for "unable to get local issuer certificate" and self-signed certificates -- these usually hint to missing root signing CAs in the certs directory. +* Several fixes for compiler (GCC, Intel C++, CLang) and autotools warnings # DOCUMENTATION * Fix table of global option to read "set softbounce" where there used to be a diff --git a/smbutil.c b/smbutil.c index a26b3496..ed6ec8a5 100644 --- a/smbutil.c +++ b/smbutil.c @@ -26,7 +26,7 @@ char versionString[] ="libntlm version 0.21"; #define AddBytes(ptr, header, buf, count) \ { \ -if (buf && count) \ +if (buf != NULL && count != 0) \ { \ SSVAL(&ptr->header.len,0,count); \ SSVAL(&ptr->header.maxlen,0,count); \ -- cgit v1.2.3