Are there any users of security/logsurfer who can verify whether it works? If not, we can simply remove the port.
Currently logsurfer is marked as not for LP64 archs. I've looked over the compiler warnings and all I found was a missing stdlib.h include and one case of truncation in regex.c. There are a lot more scary compiler warnings in regex.c, but as far as I can tell values are only cast int -> char* -> int, which is ugly but harmless. Does anybody care? Index: Makefile =================================================================== RCS file: /cvs/ports/security/logsurfer/Makefile,v retrieving revision 1.22 diff -u -p -r1.22 Makefile --- Makefile 11 Mar 2013 11:41:25 -0000 1.22 +++ Makefile 31 Aug 2015 15:14:02 -0000 @@ -3,6 +3,7 @@ COMMENT= processes logfiles and perform certain actions DISTNAME= logsurfer-1.8 +REVISION= 0 CATEGORIES= security MAINTAINER= Jakob Schlyter <ja...@openbsd.org> @@ -18,7 +19,5 @@ MASTER_SITES= http://kerryt.orcon.net.nz CONFIGURE_STYLE= gnu NO_TEST= Yes - -NOT_FOR_ARCHS= ${LP64_ARCHS} .include <bsd.port.mk> Index: patches/patch-src_readline_c =================================================================== RCS file: patches/patch-src_readline_c diff -N patches/patch-src_readline_c --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ patches/patch-src_readline_c 31 Aug 2015 15:14:02 -0000 @@ -0,0 +1,14 @@ +$OpenBSD$ +--- src/readline.c.orig Tue Aug 3 12:10:29 1999 ++++ src/readline.c Mon Aug 31 16:27:50 2015 +@@ -13,6 +13,10 @@ + + #include <stdio.h> + ++#if HAVE_STDLIB_H || STDC_HEADERS ++#include <stdlib.h> ++#endif ++ + /* ugly, but portable */ + #if HAVE_STRING_H || STDC_HEADERS + #include <string.h> Index: patches/patch-src_regex_c =================================================================== RCS file: patches/patch-src_regex_c diff -N patches/patch-src_regex_c --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ patches/patch-src_regex_c 31 Aug 2015 15:14:02 -0000 @@ -0,0 +1,12 @@ +$OpenBSD$ +--- src/regex.c.orig Tue Aug 3 12:10:29 1999 ++++ src/regex.c Mon Aug 31 16:59:38 2015 +@@ -3762,7 +3762,7 @@ re_match_2 (bufp, string1, size1, string2, size2, pos, + regstart[r] = old_regstart[r]; + + /* xx why this test? */ +- if ((int) old_regend[r] >= (int) regstart[r]) ++ if (old_regend[r] >= regstart[r]) + regend[r] = old_regend[r]; + } + } -- Christian "naddy" Weisgerber na...@mips.inka.de
