Browse code

some messages were being printed twice (bb#884)

git-svn: trunk@3725

Tomasz Kojm authored on 2008/03/20 21:10:17
Showing 2 changed files
... ...
@@ -1,3 +1,7 @@
1
+Thu Mar 20 12:16:13 CET 2008 (tk)
2
+---------------------------------
3
+  * clamdscan/client.c: some messages were being printed twice (bb#884)
4
+
1 5
 Tue Mar 18 23:55:49 EET 2008 (edwin)
2 6
 ------------------------------------
3 7
   * libclamav/scanners.c: debug message for html tempfiles
... ...
@@ -81,20 +81,18 @@ static int dsresult(int sockd, const struct optstruct *opt)
81 81
 		    *pt = 0;
82 82
 		    move_infected(buff, opt);
83 83
 		} else {
84
-		    mprintf("@Broken data format. File not %s.\n", opt_check(opt, "move") ? "moved" : "copied");
84
+		    logg("!Incorrect output from clamd. File not %s.\n", opt_check(opt, "move") ? "moved" : "copied");
85 85
 		}
86 86
 
87 87
 	    } else if(opt_check(opt, "remove")) {
88 88
 		if(!(pt = strrchr(buff, ':'))) {
89
-		    mprintf("@Broken data format. File not removed.\n");
89
+		    logg("!Incorrect output from clamd. File not removed.\n");
90 90
 		} else {
91 91
 		    *pt = 0;
92 92
 		    if(unlink(buff)) {
93
-			mprintf("~%s: Can't remove.\n", buff);
94
-			logg("~%s: Can't remove.\n", buff);
93
+			logg("!%s: Can't remove.\n", buff);
95 94
 			notremoved++;
96 95
 		    } else {
97
-			mprintf("~%s: Removed.\n", buff);
98 96
 			logg("~%s: Removed.\n", buff);
99 97
 		    }
100 98
 		}
... ...
@@ -102,7 +100,7 @@ static int dsresult(int sockd, const struct optstruct *opt)
102 102
 	}
103 103
 
104 104
 	if(strstr(buff, "ERROR\n")) {
105
-	    logg("%s", buff);
105
+	    logg("~%s", buff);
106 106
 	    waserror = 1;
107 107
 	}
108 108
     }
... ...
@@ -196,7 +194,7 @@ static int dsstream(int sockd, const struct optstruct *opt)
196 196
 	    server.sin_addr.s_addr = peer.sin_addr.s_addr;
197 197
 	    break;
198 198
 	default:
199
-	    mprintf("^Unexpected socket type: %d.\n", peer.sin_family);
199
+	    logg("^Unexpected socket type: %d.\n", peer.sin_family);
200 200
 	    return -1;
201 201
     }
202 202