Bug 195019 - security/snort -- missing required link library name for APPID option
Summary: security/snort -- missing required link library name for APPID option
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Ryan Steinmetz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-11-14 22:41 UTC by Bill Meeks
Modified: 2015-01-06 01:33 UTC (History)
1 user (show)

See Also:
bmeeks8: maintainer-feedback? (zi)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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 freebsd_triage 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 freebsd_triage 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