Bug 174379 - Temporal fix to include/sasl.h
Summary: Temporal fix to include/sasl.h
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Hajimu UMEMOTO
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-11 22:10 UTC by sakaue.mamoru
Modified: 2012-12-25 17:15 UTC (History)
0 users

See Also:


Attachments
file.diff (707 bytes, patch)
2012-12-11 22:10 UTC, sakaue.mamoru
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description sakaue.mamoru 2012-12-11 22:10:00 UTC
The current version (cyrus-sasl-2.1.26) started to invoke "size_t" in include/sasl.h started to invoke but lacks a line to include stddef.h or stdlib.h.
This is a platform-independent bug and an equivalent issue has already been reported to the forum in the developer site by other person, but it may take time for the new fixed version to be up.
So I submit an equivalent patch for a temporal countermeasure.
This patch is expected to be obsolete for the next version.

Fix: Apply the attached patch.

security:cyrus-sasl2.patch
===========================================
How-To-Repeat: In the current ports tree (updated at 4:08, 11 Dec 2012, +9:00), execute

make -C /usr/ports/deskutils/kdepimlibs4

Then the build fails by complaining about the above matter.
Here the version of the example is kdepimlibs-4.8.4_1.
Comment 1 Tilman Keskinoz freebsd_committer freebsd_triage 2012-12-17 16:46:53 UTC
Responsible Changed
From-To: freebsd-ports-bugs->ume

over to maintainer
Comment 2 Hajimu UMEMOTO freebsd_committer freebsd_triage 2012-12-18 06:20:24 UTC
Hi,

> This is a platform-independent bug and an equivalent issue has
> already been reported to the forum in the developer site by other
> person, but it may take time for the new fixed version to be up.

Yes, I think it's a platform-independent bug, too.
If you mean that the forum is cyrus-sasl list, it's me. :-)

> So I submit an equivalent patch for a temporal countermeasure.
> This patch is expected to be obsolete for the next version.

The problem in deskutils/kdepimlibs4 as you mentioned was already
fixed.
I'm waiting for the answer from the author.  Whether to modify the
cyrus-sasl2 port is depending on the response from the author.

Sincerely,

--
Hajimu UMEMOTO
ume@mahoroba.org  ume@{,jp.}FreeBSD.org
http://www.mahoroba.org/~ume/
Comment 3 dfilter service freebsd_committer freebsd_triage 2012-12-25 17:02:18 UTC
Author: ume
Date: Tue Dec 25 17:02:10 2012
New Revision: 309488
URL: http://svnweb.freebsd.org/changeset/ports/309488

Log:
  Add upstream commit to define size_t in sasl.h.
  
  PR:		ports/174379, ports/174692
  Submitted by:	rakuco
  Obtained from:	http://git.cyrusimap.org/cyrus-sasl/commit/include/sasl.h?id=67a188693796a14e3a76ac603104807fbbfddfc4

Added:
  head/security/cyrus-sasl2/files/patch-include::sasl.h   (contents, props changed)
Modified:
  head/security/cyrus-sasl2/Makefile

Modified: head/security/cyrus-sasl2/Makefile
==============================================================================
--- head/security/cyrus-sasl2/Makefile	Tue Dec 25 16:28:34 2012	(r309487)
+++ head/security/cyrus-sasl2/Makefile	Tue Dec 25 17:02:10 2012	(r309488)
@@ -2,7 +2,7 @@
 
 PORTNAME=	cyrus-sasl
 PORTVERSION=	2.1.26
-PORTREVISION=	1
+PORTREVISION=	2
 CATEGORIES=	security ipv6
 MASTER_SITES=	ftp://ftp.cyrusimap.org/cyrus-sasl/
 

Added: head/security/cyrus-sasl2/files/patch-include::sasl.h
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/security/cyrus-sasl2/files/patch-include::sasl.h	Tue Dec 25 17:02:10 2012	(r309488)
@@ -0,0 +1,23 @@
+From 67a188693796a14e3a76ac603104807fbbfddfc4 Mon Sep 17 00:00:00 2001
+From: Ken Murchison <murch@andrew.cmu.edu>
+Date: Thu, 20 Dec 2012 23:14:50 +0000
+Subject: sasl.h: #include <stddef.h> for size_t on NetBSD
+
+---
+(limited to 'include/sasl.h')
+
+diff --git a/include/sasl.h b/include/sasl.h
+index fef4d51..8b8a63f 100755
+--- include/sasl.h
++++ include/sasl.h
+@@ -121,6 +121,8 @@
+ #ifndef SASL_H
+ #define SASL_H 1
+ 
++#include <stddef.h>  /* For size_t */
++
+ /* Keep in sync with win32/common.mak */
+ #define SASL_VERSION_MAJOR 2
+ #define SASL_VERSION_MINOR 1
+--
+cgit v0.9.0.2
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 4 Hajimu UMEMOTO freebsd_committer freebsd_triage 2012-12-25 17:13:27 UTC
State Changed
From-To: open->closed

Upstream fix was committed. 
Thank you for your cooperation.