pkg/27337: mail/clamav Bad format string in clamav-milter

[MORI Kouji <moriko%hh.iij4u.or.jp@localhost>]

        Note: There was a bad value `<[' for the field `>Confidential:'.
        It was set to the default value of `yes'.


>Number:         27337
>Category:       pkg
>Synopsis:       mail/clamav Bad format string in clamav-milter
>Confidential:   yes
>Severity:       serious
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Oct 22 08:47:06 UTC 2004
>Closed-Date:
>Last-Modified:
>Originator:     Koji Mori
>Release:        NetBSD 1.6.2_STABLE
>Organization:
>Environment:
>Description:
in clamav-0.80, clamav-milter/clamav-milter.c has follow expressions.

   fprintf(sendmail, _("The message %1$s sent from %2$s to\n"), sendmailId, 
sender);

But NetBSD's fprintf(3) doesn't support "%1$s".

>How-To-Repeat:
>Fix:
--- clamav-milter/clamav-milter.c.orig  Fri Oct 22 16:09:05 2004
+++ clamav-milter/clamav-milter.c       Fri Oct 22 16:19:28 2004
@@ -2691,7 +2691,7 @@
                                                 * the postmaster, so include
                                                 * some useful information
                                                 */
-                                               fprintf(sendmail, _("The 
message %1$s sent from %2$s to\n"),
+                                               fprintf(sendmail, _("The 
message %s sent from %s to\n"),
                                                        sendmailId, sender);
                                        else
                                                fprintf(sendmail, _("A message 
sent from %s to\n"),
@@ -2706,7 +2706,7 @@
                                                        fprintf(sendmail, 
_("\nThe message in question has been quarantined as %s\n"), 
privdata->filename);
 
                                        if(hflag) {
-                                               fprintf(sendmail, _("\nThe 
message was received by %1$s from %2$s via %3$s\n\n"),
+                                               fprintf(sendmail, _("\nThe 
message was received by %s from %s via %s\n\n"),
                                                        smfi_getsymval(ctx, 
"j"), sender,
                                                        smfi_getsymval(ctx, 
"_"));
                                                fputs(_("For your information, 
the original message headers were:\n\n"), sendmail);
@@ -3697,7 +3697,7 @@
        if(link(privdata->filename, newname) < 0) {
                perror(newname);
                if(use_syslog)
-                       syslog(LOG_WARNING, _("Can't rename %1$s to %2$s"),
+                       syslog(LOG_WARNING, _("Can't rename %s to %s"),
                                privdata->filename, newname);
                free(newname);
                return -1;

>Release-Note:
>Audit-Trail:
>Unformatted: