Bug 229692 - net/openvswitch : Fixing crash on FreeBSD
Summary: net/openvswitch : Fixing crash on FreeBSD
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Olivier Cochard
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-11 09:08 UTC by Olivier Cochard
Modified: 2018-07-11 12:43 UTC (History)
1 user (show)

See Also:
koue: maintainer-feedback+


Attachments
patch fixing openvswitch (1.66 KB, patch)
2018-07-11 09:08 UTC, Olivier Cochard
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Olivier Cochard freebsd_committer freebsd_triage 2018-07-11 09:08:45 UTC
Created attachment 195046 [details]
patch fixing openvswitch

openvswitch crashs on FreeBSD, cf https://github.com/openvswitch/ovs-issues/issues/152.

Here is the patch fixing this problem.
Comment 1 Olivier Cochard freebsd_committer freebsd_triage 2018-07-11 10:01:40 UTC
Just to be sure, because there is no comment :-)

By changing the "maintainer-feedback" flags, do you mean:
- "I've seen this PR, and  please don't maintainer-timeout while I'm testing it"
- or "I'm agree with the patch, please commit it"
Comment 2 Nikola Kolev 2018-07-11 10:27:51 UTC
I'm agree with the patch, please commit it.

Thanks :)
Comment 3 commit-hook freebsd_committer freebsd_triage 2018-07-11 12:42:30 UTC
A commit references this bug:

Author: olivier
Date: Wed Jul 11 12:42:02 UTC 2018
New revision: 474441
URL: https://svnweb.freebsd.org/changeset/ports/474441

Log:
  Fix crash of ovs-vswitchd with 'add-br' command.
  Reference: https://github.com/openvswitch/ovs-issues/issues/152

  PR:		229692
  Approved by:	Nikola Kolev <koue@chaosophia.net>(maintainer)
  Sponsored by:	Orange

Changes:
  head/net/openvswitch/Makefile
  head/net/openvswitch/files/patch-lib_netdev-bsd.c
Comment 4 Olivier Cochard freebsd_committer freebsd_triage 2018-07-11 12:43:48 UTC
Committed (tested on 10.4-amd64 and current-amd64).