]> git.ipfire.org Git - thirdparty/sarg.git/blobdiff - smartfilter.c
Strengthen the making of the report file name to drastically limit the characters...
[thirdparty/sarg.git] / smartfilter.c
index 1ecb57ee56dd3c74b97b290a33e54bc98547aee2..d0d281f611ea61021e4a182d795a2e07bfa01047 100644 (file)
@@ -32,14 +32,13 @@ void smartfilter_report(void)
 
    FILE *fp_in = NULL, *fp_ou = NULL, *fp_user = NULL;
 
+   char buf[MAXLEN];
    char url[MAXLEN];
    char csort[255];
    char smart_in[MAXLEN];
    char smart_ou[MAXLEN];
-   char per[MAXLEN];
    char sites[MAXLEN];
    char report[MAXLEN];
-   char period[100];
    char ip[MAXLEN];
    char user[MAXLEN];
    char ouser[MAXLEN];
@@ -56,46 +55,34 @@ void smartfilter_report(void)
 
    ouser[0]='\0';
 
-   sprintf(smartheader,"%s",_("SmartFilter"));
+   strcpy(smartheader,_("SmartFilter"));
    strup(smartheader);
 
-   sprintf(smart_in,"%s/smartfilter.unsort",dirname);
-   sprintf(sites,"%s/sarg-sites",dirname);
-   sprintf(smart_ou,"%s/smartfilter.log",dirname);
-   sprintf(per,"%s/sarg-period",dirname);
-   sprintf(report,"%s/smartfilter.html",dirname);
-
-   if ((fp_in = fopen(per, "r")) == 0) {
-      fprintf(stderr, "SARG: (smartfilter) %s: %s\n",_("Cannot open file"),per);
-      exit(1);
-   }
-
-   if (!fgets(period,sizeof(period),fp_in)) {
-      debuga(_("(smartfilter) read error in %s"),per);
-      exit(1);
-   }
-   fclose(fp_in);
+   sprintf(smart_in,"%s/smartfilter.unsort",outdirname);
+   sprintf(sites,"%s/sarg-sites",outdirname);
+   sprintf(smart_ou,"%s/smartfilter.log",outdirname);
+   sprintf(report,"%s/smartfilter.html",outdirname);
 
    if (snprintf(csort,sizeof(csort),"sort -n -k 1,1 -k 2,2 -k 3,3 -o \"%s\" \"%s\"",smart_ou,smart_in)>=sizeof(csort)) {
-      debuga(_("cannot build the sort command to sort file %s"),smart_in);
-      exit(1);
+      debuga(_("cannot build the sort command to sort file %s\n"),smart_in);
+      exit(EXIT_FAILURE);
    }
    cstatus=system(csort);
    if (!WIFEXITED(cstatus) || WEXITSTATUS(cstatus)) {
-      debuga(_("sort command return status %d"),WEXITSTATUS(cstatus));
-      debuga(_("sort command: %s"),csort);
-      exit(1);
+      debuga(_("sort command return status %d\n"),WEXITSTATUS(cstatus));
+      debuga(_("sort command: %s\n"),csort);
+      exit(EXIT_FAILURE);
    }
    if((fp_in=fopen(smart_ou,"r"))==NULL) {
-      fprintf(stderr, "SARG: (smartfilter) %s: %s\n",_("Cannot open log file"),smart_ou);
-      debuga(_("sort command: %s"),csort);
-      exit(1);
+      debuga(_("(smartfilter) Cannot open log file %s\n"),smart_ou);
+      debuga(_("sort command: %s\n"),csort);
+      exit(EXIT_FAILURE);
    }
    unlink(smart_in);
 
    if((fp_ou=fopen(report,"w"))==NULL) {
-     fprintf(stderr, "SARG: (smartfilter) %s: %s\n",_("Cannot open log file"),report);
-     exit(1);
+     debuga(_("(smartfilter) Cannot open log file %s\n"),report);
+     exit(EXIT_FAILURE);
    }
 
    fprintf(fp_ou, "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\" \"http://www.w3.org/TR/html4/loose.dtd\">\n<html>\n<head>\n  <meta http-equiv=\"Content-Type\" content=\"text/html; charset=%s\">\n",CharSet);
@@ -106,7 +93,9 @@ void smartfilter_report(void)
    write_logo_image(fp_ou);
 
    fprintf(fp_ou,"<tr><th align=\"center\"><b><font color=\"%s\" size=\"+1\">%s</font></b></th></tr>\n",TiColor,Title);
-   fprintf(fp_ou,"<tr><td align=\"center\" bgcolor=\"%s\"><font size=\"%s\">%s: %s</font></td></tr>\n",HeaderBgColor,FontSize,_("Period"),period);
+   fprintf(fp_ou,"<tr><td align=\"center\" bgcolor=\"%s\"><font size=\"%s\">",HeaderBgColor,FontSize);
+   fprintf(fp_ou,_("Period: %s"),period.html);
+   fputs("</font></td></tr>\n",fp_ou);
    fprintf(fp_ou,"<tr><th bgcolor=\"%s\" align=\"center\"><font size=\"%s\">%s</font></th></tr>\n",HeaderBgColor,FontSize,_("SmartFilter"));
    fputs("</table></div>\n",fp_ou);
 
@@ -121,18 +110,18 @@ void smartfilter_report(void)
       if (getword(user,sizeof(user),&gwarea,'\t')<0 || getword(data,sizeof(data),&gwarea,'\t')<0 ||
           getword(hora,sizeof(hora),&gwarea,'\t')<0 || getword(ip,sizeof(ip),&gwarea,'\t')<0 ||
           getword(url,sizeof(url),&gwarea,'\t')<0 || getword(smartcat,sizeof(smartcat),&gwarea,'\n')<0) {
-         debuga(_("There is e a broken record or garbage in file %s"),smart_ou);
-         exit(1);
+         debuga(_("There is a broken record or garbage in file %s\n"),smart_ou);
+         exit(EXIT_FAILURE);
       }
 
       uinfo=userinfo_find_from_id(user);
       if (!uinfo) {
-         debuga(_("Unknown user ID %s in file %s"),user,smart_ou);
-         exit(1);
+         debuga(_("Unknown user ID %s in file %s\n"),user,smart_ou);
+         exit(EXIT_FAILURE);
       }
       if(strcmp(ouser,user) != 0) {
          strcpy(ouser,user);
-         sprintf(smartuser,"%s/denied_%s.html",dirname,uinfo->filename);
+         sprintf(smartuser,"%s/denied_%s.html",outdirname,uinfo->filename);
          if(fuser) {
             fuser=0;
             fputs("</table>\n",fp_user);
@@ -144,8 +133,8 @@ void smartfilter_report(void)
             fclose(fp_user);
          }
          if ((fp_user = fopen(smartuser, "a")) == 0) {
-            fprintf(stderr, "SARG: (smartfilter) %s: %s\n",_("Cannot open file"),smartuser);
-            exit(1);
+            debuga(_("(smartfilter) Cannot open file %s\n"),smartuser);
+            exit(EXIT_FAILURE);
          }
          fuser=1;
 
@@ -172,7 +161,9 @@ void smartfilter_report(void)
          fputs("<div align=\"center\"><table cellpadding=\"0\" cellspacing=\"0\">\n",fp_user);
          if(LogoImage[0]!='\0') fprintf(fp_user,"<tr><th align=left><img src=\"%s\" border=\"0\" align=\"absmiddle\" width=\"%s\" height=\"%s\"><font color=\"%s\">%s</font>\n",LogoImage,Width,Height,LogoTextColor,LogoText);
          fprintf(fp_user,"<tr><th align=\"center\"><b><font color=\"%s\" size=\"+1\">%s</font></b></th></tr>\n",TiColor,Title);
-         fprintf(fp_user,"<tr><td align=center bgcolor=%s><font size=%s>%s: %s</font></td></tr>\n",HeaderBgColor,FontSize,_("Period"),period);
+         fputs("<tr><td align=center bgcolor=\"%s\"><font size=\"%s\">",fp_user);
+         fprintf(fp_user,_("Period: %s"),period.html);
+         fputs("</font></td></tr>\n",fp_user);
          fprintf(fp_user,"<tr><td align=\"center\" bgcolor=\"%s\"><font size=\"%s\">%s:</font><font size=\"%s\"> %s</font></td></tr>\n",HeaderBgColor,FontSize,_("User"),FontSize,uinfo->label);
          fputs("</table></div>\n",fp_user);
          fputs("<div align=\"center\"><table cellpadding=0 cellspacing=2>\n",fp_user);