[packages/clamav] - rel 2; don't trash stderr with meaningless errors, log these instead

arekm arekm at pld-linux.org
Sat Mar 15 12:50:47 CET 2014


commit 32b1d1ee3a842ed5b2fc0b1b75cbfd1693b20899
Author: Arkadiusz Miśkiewicz <arekm at maven.pl>
Date:   Sat Mar 15 12:50:44 2014 +0100

    - rel 2; don't trash stderr with meaningless errors, log these instead

 clamav-notify-error.patch | 91 +++++++++++++++++++++++++++++++++++++++++++++++
 clamav.spec               |  4 ++-
 2 files changed, 94 insertions(+), 1 deletion(-)
---
diff --git a/clamav.spec b/clamav.spec
index 989a4a2..e95c530 100644
--- a/clamav.spec
+++ b/clamav.spec
@@ -14,7 +14,7 @@ Summary:	An anti-virus utility for Unix
 Summary(pl.UTF-8):	Narzędzie antywirusowe dla Uniksów
 Name:		clamav
 Version:	0.98.1
-Release:	1
+Release:	2
 License:	GPL v2+
 Group:		Daemons
 Source0:	http://downloads.sourceforge.net/clamav/%{name}-%{version}.tar.gz
@@ -32,6 +32,7 @@ Patch0:		%{name}-pld_config.patch
 Patch1:		%{name}-nolibs.patch
 Patch2:		am-nosilentrules.patch
 Patch3:		ac2.68.patch
+Patch4:		%{name}-notify-error.patch
 URL:		http://www.clamav.net/
 BuildRequires:	autoconf
 BuildRequires:	automake
@@ -163,6 +164,7 @@ Biblioteki statyczne clamav.
 %patch1 -p1
 %patch2 -p1
 %patch3 -p1
+%patch4 -p1
 
 %build
 %{__libtoolize}
diff --git a/clamav-notify-error.patch b/clamav-notify-error.patch
new file mode 100644
index 0000000..380a56f
--- /dev/null
+++ b/clamav-notify-error.patch
@@ -0,0 +1,91 @@
+--- clamav-0.98.1/freshclam/notify.c.org	2014-03-15 12:30:42.386370064 +0100
++++ clamav-0.98.1/freshclam/notify.c	2014-03-15 12:37:47.097443450 +0100
+@@ -82,8 +82,8 @@
+ 
+         if ((sockd = socket (AF_UNIX, SOCK_STREAM, 0)) < 0)
+         {
+-            perror ("socket()");
+-            logg ("^Clamd was NOT notified: Can't create socket endpoint for %s\n", opt->strarg);
++            logg ("^Clamd was NOT notified: Can't create socket endpoint for %s: %s\n",
++                opt->strarg, strerror(errno));
+             optfree (opts);
+             return -1;
+         }
+@@ -92,9 +92,9 @@
+             (sockd, (struct sockaddr *) &server,
+              sizeof (struct sockaddr_un)) < 0)
+         {
+-            perror ("connect()");
++            logg ("^Clamd was NOT notified: Can't connect to clamd through %s: %s\n",
++                opt->strarg, strerror(errno));
+             closesocket (sockd);
+-            logg ("^Clamd was NOT notified: Can't connect to clamd through %s\n", opt->strarg);
+             optfree (opts);
+             return -11;
+         }
+@@ -137,8 +137,7 @@
+ 
+         if ((sockd = socket (res->ai_family, SOCK_STREAM, 0)) < 0)
+         {
+-            perror ("socket()");
+-            logg ("!%s: Can't create TCP socket\n", option);
++            logg ("!%s: Can't create TCP socket: %s\n", option, strerror(errno));
+             optfree (opts);
+             freeaddrinfo (res);
+             return -1;
+@@ -146,10 +145,9 @@
+ 
+         if (connect (sockd, res->ai_addr, res->ai_addrlen) == -1)
+         {
+-            perror ("connect()");
++            logg ("!%s: Can't connect to clamd on %s:%s: %s\n", option,
++                  addr ? addr : "localhost", port, strerror(errno));
+             closesocket (sockd);
+-            logg ("!%s: Can't connect to clamd on %s:%s\n", option,
+-                  addr ? addr : "localhost", port);
+             optfree (opts);
+             freeaddrinfo (res);
+             return -1;
+@@ -160,8 +158,7 @@
+ 
+         if ((sockd = socket (AF_INET, SOCK_STREAM, 0)) < 0)
+         {
+-            perror ("socket()");
+-            logg ("!%s: Can't create TCP socket\n", option);
++            logg ("!%s: Can't create TCP socket: %s\n", option, strerror(errno));
+             optfree (opts);
+             return -1;
+         }
+@@ -173,8 +170,8 @@
+         {
+             if ((he = gethostbyname (opt->strarg)) == 0)
+             {
+-                perror ("gethostbyname()");
+-                logg ("^Clamd was NOT notified: Can't resolve hostname '%s'\n", opt->strarg);
++                logg ("^Clamd was NOT notified: Can't resolve hostname '%s': %s\n",
++                    opt->strarg, strerror(errno));
+                 optfree (opts);
+                 closesocket (sockd);
+                 return -1;
+@@ -189,9 +186,9 @@
+             (sockd, (struct sockaddr *) &server2,
+              sizeof (struct sockaddr_in)) < 0)
+         {
+-            perror ("connect()");
++            logg ("^Clamd was NOT notified: Can't connect to clamd on %s:%d: %s\n",
++                inet_ntoa (server2.sin_addr), ntohs (server2.sin_port), strerror(errno));
+             closesocket (sockd);
+-            logg ("^Clamd was NOT notified: Can't connect to clamd on %s:%d\n", inet_ntoa (server2.sin_addr), ntohs (server2.sin_port));
+             optfree (opts);
+             return -1;
+         }
+@@ -222,8 +219,7 @@
+ 
+     if (sendln (sockd, "RELOAD", 7) < 0)
+     {
+-        perror ("send()");
+-        logg ("!NotifyClamd: Could not write to clamd socket\n");
++        logg ("!NotifyClamd: Could not write to clamd socket: %s\n", strerror(errno));
+         closesocket (sockd);
+         return 1;
+     }
================================================================

---- gitweb:

http://git.pld-linux.org/gitweb.cgi/packages/clamav.git/commitdiff/32b1d1ee3a842ed5b2fc0b1b75cbfd1693b20899



More information about the pld-cvs-commit mailing list