Bug 199442

Summary: security/strongswan: fix smp_plugin_create() returned NULL error
Product: Ports & Packages Reporter: Franco Fichtner <franco>
Component: Individual Port(s)Assignee: Mark Felder <feld>
Status: Closed FIXED    
Severity: Affects Only Me CC: brnrd, feld, strongswan
Priority: --- Flags: strongswan: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
smp fix + revision bump
none
poudriere build log
none
regenerated patch file
none
regenerated patch file 2
none
regenerated patch file 3
none
svn diff for security/strongswan
none
svn diff for security/strongswan
none
svn diff for security/strongswan
none
svn diff for security/strongswan
none
poudriere build log
none
svn diff for security/strongswan none

Description Franco Fichtner 2015-04-14 09:04:10 UTC
Created attachment 155587 [details]
smp fix + revision bump

via https://github.com/opnsense/ports/issues/8

Possible upstream candidate although the fix is not portable (due to strlcpy).
Comment 1 Francois ten Krooden 2015-04-15 05:04:08 UTC
I'm happy.  Will pass the info about the issue on upstream as well.
Comment 2 Franco Fichtner 2015-04-15 07:35:26 UTC
Created attachment 155604 [details]
poudriere build log
Comment 3 Franco Fichtner 2015-04-15 07:36:52 UTC
Created attachment 155605 [details]
regenerated patch file
Comment 4 Franco Fichtner 2015-04-15 07:37:13 UTC
Created attachment 155606 [details]
regenerated patch file 2
Comment 5 Franco Fichtner 2015-04-15 07:37:33 UTC
Created attachment 155607 [details]
regenerated patch file 3
Comment 6 Franco Fichtner 2015-04-15 07:38:41 UTC
Updated all patches using "makepatch" since portlint complained. `portlint -a' is now silent.  Regenerated files attached.  Also added compressed poudriere build log.
Comment 7 Bernard Spil freebsd_committer freebsd_triage 2015-04-15 07:58:42 UTC
Created attachment 155609 [details]
svn diff for security/strongswan

Merged all 4 patch-files into a single svn diff
Comment 8 Bernard Spil freebsd_committer freebsd_triage 2015-04-15 08:08:48 UTC
Created attachment 155611 [details]
svn diff for security/strongswan

The 2 deletes for the rename were missing...  Next try
Comment 9 Bernard Spil freebsd_committer freebsd_triage 2015-04-15 08:14:36 UTC
Created attachment 155612 [details]
svn diff for security/strongswan
Comment 10 Bernard Spil freebsd_committer freebsd_triage 2015-04-15 08:55:01 UTC
Created attachment 155614 [details]
svn diff for security/strongswan

Now including the smp patch.
Comment 11 Franco Fichtner 2015-04-15 09:05:54 UTC
Created attachment 155617 [details]
poudriere build log

refreshing build log to obsolete files
Comment 12 Bernard Spil freebsd_committer freebsd_triage 2015-04-15 12:24:17 UTC
Created attachment 155623 [details]
svn diff for security/strongswan

Should now all be there. Renaming and additions.
Comment 13 Franco Fichtner 2015-04-15 12:29:38 UTC
Looking good, thanks Bernard :)
Comment 14 Mark Felder freebsd_committer freebsd_triage 2015-04-15 16:43:34 UTC
I'll take it
Comment 15 commit-hook freebsd_committer freebsd_triage 2015-04-16 12:56:39 UTC
A commit references this bug:

Author: feld
Date: Thu Apr 16 12:55:40 UTC 2015
New revision: 384108
URL: https://svnweb.freebsd.org/changeset/ports/384108

Log:
  Add patches to fix Strongswan Management Protocol

  SMP is an XML control interface for Strongswan used by pfSense and
  Opnsense. SMP has been deprecated by upstream since 5.2.0 in favor of a
  newer IPC mechanism called VICI. As a result upstream is not motivated
  to take patches for SMP, and this uses non-portable strlcpy anyway.

  The code has not been deleted from the project and if we can bludgeon it
  into a working state I see no harm.

  PR:		199442

Changes:
  head/security/strongswan/Makefile
  head/security/strongswan/files/patch-conf_Makefile.in
  head/security/strongswan/files/patch-conf__Makefile.in
  head/security/strongswan/files/patch-src__starter__Makefile.in
  head/security/strongswan/files/patch-src_libcharon_plugins_smp_smp.c
  head/security/strongswan/files/patch-src_starter_Makefile.in
Comment 16 Mark Felder freebsd_committer freebsd_triage 2015-04-16 13:00:19 UTC
Committed, thanks!