Bug 195019

Summary: security/snort -- missing required link library name for APPID option
Product: Ports & Packages Reporter: Bill Meeks <bmeeks8>
Component: Individual Port(s)Assignee: Ryan Steinmetz <zi>
Status: Closed FIXED    
Severity: Affects Many People CC: bmeeks8
Priority: --- Flags: bmeeks8: maintainer-feedback? (zi)
Version: Latest   
Hardware: Any   
OS: Any   

Description Bill Meeks 2014-11-14 22:41:56 UTC
The link library name for the APPID option used to enable the new OpenAppID feature in Snort 2.9.7.0 in missing in the Makefile.

This line should enable linking with the luajit-5.1 library required for APPID support, but notice the required library name is missing:

APPID_CONFIGURE_ENV+=	luajit_CFLAGS="-I${LOCALBASE}/include/luajit-2.0" \
			luajit_LIBS="-L${LOCALBASE}/lib"


This is the corrected line with the required library specified:

APPID_CONFIGURE_ENV+=	luajit_CFLAGS="-I${LOCALBASE}/include/luajit-2.0" \
			luajit_LIBS="-L${LOCALBASE}/lib -lluajit-5.1"
Comment 1 Bugzilla Automation freebsd_committer 2014-11-14 22:41:56 UTC
Auto-assigned to maintainer zi@FreeBSD.org
Comment 2 Bill Meeks 2014-11-14 22:43:53 UTC
Without the required library name specified, Snort will successfully compile but you get a runtime error with undefined symbols when you attempt to enable the APPID feature in snort.conf.
Comment 3 commit-hook freebsd_committer 2015-01-06 01:33:55 UTC
A commit references this bug:

Author: zi
Date: Tue Jan  6 01:33:34 UTC 2015
New revision: 376381
URL: https://svnweb.freebsd.org/changeset/ports/376381

Log:
  - Fix runtime issue with APPID enabled

  PR:		195019
  Submitted by:	Bill Meeks <bmeeks8@bellsouth.net>

Changes:
  head/security/snort/Makefile