Bug 204845

Summary: comms/efax: Core dumps on amd64
Product: Ports & Packages Reporter: yama
Component: Individual Port(s)Assignee: Thomas Zander <riggs>
Status: Closed FIXED    
Severity: Affects Many People CC: riggs
Priority: --- Keywords: crash, easy, needs-qa, patch, patch-ready
Version: LatestFlags: riggs: merge-quarterly+
Hardware: amd64   
OS: Any   
Attachments:
Description Flags
A patch file for solution. koobs: maintainer-approval+

Description yama 2015-11-27 12:59:12 UTC
Created attachment 163581 [details]
A patch file for solution.

I install comms/efax using pkg command on FreeBSD10.1/amd64. The program stops with core dump.

How to reproduce:
* Edit a configuration file /usr/local/etc/efax.rc.
* Run a command as bellow.
    fax send FAX-NUMBER FILENAME

Reason:
A variable (va_list ap) is broken.

Solution:
Apply patch. This patch initializes the variable ap every time in a for loop.
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2015-11-27 13:17:46 UTC
Comment on attachment 163581 [details]
A patch file for solution.

Port is un-maintained, implicit approval
Comment 2 commit-hook freebsd_committer freebsd_triage 2015-11-29 16:52:16 UTC
A commit references this bug:

Author: riggs
Date: Sun Nov 29 16:52:06 UTC 2015
New revision: 402606
URL: https://svnweb.freebsd.org/changeset/ports/402606

Log:
  Fix runtime segfault due to uninitialised variable, bump PORTREVISION

  While on it:
  - Pet portlint: regenerate patches using 'make makepatch'
  - Add LICENSE

  PR:		204845
  Submitted by:	yama@jasper.co.jp
  MFH:		2015Q4 (ports-secteam fix blanket)

Changes:
  head/comms/efax/Makefile
  head/comms/efax/files/patch-Makefile
  head/comms/efax/files/patch-efaxmsg.c
  head/comms/efax/files/patch-fax
  head/comms/efax/files/patch-src_Makefile
  head/comms/efax/files/patch-src_fax
Comment 3 commit-hook freebsd_committer freebsd_triage 2015-11-29 16:56:18 UTC
A commit references this bug:

Author: riggs
Date: Sun Nov 29 16:56:06 UTC 2015
New revision: 402607
URL: https://svnweb.freebsd.org/changeset/ports/402607

Log:
  MFH: r402606

  Fix runtime segfault due to uninitialised variable, bump PORTREVISION

  While on it:
  - Pet portlint: regenerate patches using 'make makepatch'
  - Add LICENSE

  PR:		204845
  Submitted by:	yama@jasper.co.jp
  Approved by:	ports-secteam (fix blanket)

Changes:
_U  branches/2015Q4/
  branches/2015Q4/comms/efax/Makefile
  branches/2015Q4/comms/efax/files/patch-Makefile
  branches/2015Q4/comms/efax/files/patch-efaxmsg.c
  branches/2015Q4/comms/efax/files/patch-fax
  branches/2015Q4/comms/efax/files/patch-src_Makefile
  branches/2015Q4/comms/efax/files/patch-src_fax