From nobody@FreeBSD.org  Wed Jun 24 18:45:33 2009
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id C0DF4106567C
	for <freebsd-gnats-submit@FreeBSD.org>; Wed, 24 Jun 2009 18:45:33 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (www.freebsd.org [IPv6:2001:4f8:fff6::21])
	by mx1.freebsd.org (Postfix) with ESMTP id AE5778FC28
	for <freebsd-gnats-submit@FreeBSD.org>; Wed, 24 Jun 2009 18:45:33 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (localhost [127.0.0.1])
	by www.freebsd.org (8.14.3/8.14.3) with ESMTP id n5OIjXT6072375
	for <freebsd-gnats-submit@FreeBSD.org>; Wed, 24 Jun 2009 18:45:33 GMT
	(envelope-from nobody@www.freebsd.org)
Received: (from nobody@localhost)
	by www.freebsd.org (8.14.3/8.14.3/Submit) id n5OIjXQG072374;
	Wed, 24 Jun 2009 18:45:33 GMT
	(envelope-from nobody)
Message-Id: <200906241845.n5OIjXQG072374@www.freebsd.org>
Date: Wed, 24 Jun 2009 18:45:33 GMT
From: Boris Lytochkin <lytboris@gmail.com>
To: freebsd-gnats-submit@FreeBSD.org
Subject: libpdel is broken on CURRENT after June,9 
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         136012
>Category:       ports
>Synopsis:       devel/libpdel is broken on CURRENT after June,9
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    wxs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed Jun 24 18:50:01 UTC 2009
>Closed-Date:    Mon Jun 29 15:25:44 UTC 2009
>Last-Modified:  Mon Jun 29 15:30:06 UTC 2009
>Originator:     Boris Lytochkin
>Release:        800099
>Organization:
Yandex
>Environment:
FreeBSD  8.0-CURRENT FreeBSD 8.0-CURRENT #0: Tue Jun 23 22:39:46 MSD 2009     root@:/opt/obj/usr/src/sys/GENERIC  i386

>Description:
*LEN macros that were deleted in 1.31 commit are used both in devel/libpdel and net/mpd4 ports - this breaks building of them.
>How-To-Repeat:
update to CURRENT after June, 9
make -C /usr/ports/devel/libpdel clean all

>Fix:
All *LEN macros should be replaced with appropriate *SIZ macros.


Patch attached with submission follows:

--- ppp/ppp_l2tp_ctrl.c.orig	2005-01-22 00:02:07.000000000 +0300
+++ ppp/ppp_l2tp_ctrl.c	2009-06-24 22:00:15.000000000 +0400
@@ -200,7 +200,7 @@
 	u_int16_t		peer_id;		/* peer session id */
 	struct ppp_log		*log;			/* log */
 	ng_ID_t			node_id;		/* tee node id */
-	char			hook[NG_HOOKLEN + 1];	/* session hook name */
+	char			hook[NG_HOOKSIZ];	/* session hook name */
 	void			*link_cookie;		/* opaque link cookie */
 	u_int16_t		result;			/* close result code */
 	u_int16_t		error;			/* close error code */
@@ -608,7 +608,7 @@
 
 	/* Done */
 	*nodep = ctrl->node_id;
-	strlcpy(hook, NG_L2TP_HOOK_LOWER, NG_HOOKLEN + 1);
+	strlcpy(hook, NG_L2TP_HOOK_LOWER, NG_HOOKSIZ);
 	return (ctrl);
 
 fail:
@@ -1860,7 +1860,7 @@
 	    struct ng_mesg msg;
 	} buf;
 	struct ng_mesg *const msg = &buf.msg;
-	char raddr[NG_PATHLEN + 1];
+	char raddr[NG_PATHSIZ];
 	int len;
 
 	/* Read netgraph control message */
--- ppp/ppp_l2tp_ctrl.h.orig	2005-01-22 00:02:07.000000000 +0300
+++ ppp/ppp_l2tp_ctrl.h	2009-06-24 22:00:15.000000000 +0400
@@ -201,7 +201,7 @@
  *	peer_id	Unique identifier for peer (used for tie-breakers)
  *	initiate Whether to send a SCCRQ or just wait for one
  *	nodep	Pointer to netgraph node ID variable
- *	hook	Buffer for hook on L2TP netgraph node (size >= NG_HOOKLEN + 1)
+ *	hook	Buffer for hook on L2TP netgraph node (size >= NG_HOOKSIZ)
  *	avps	List of AVP's to include in the associated
  *		Start-Control-Connection-Request or
  *		Start-Control-Connection-Reply control message.
--- ppp/ppp_l2tp_server.c.orig	2005-01-22 00:02:07.000000000 +0300
+++ ppp/ppp_l2tp_server.c	2009-06-24 22:00:15.000000000 +0400
@@ -103,7 +103,7 @@
 	struct ppp_channel		*chan;		/* pointer to channel */
 	struct ppp_auth_config		auth;		/* auth config */
 	char				node[32];		/* node path */
-	char				hook[NG_HOOKLEN + 1];	/* node hook */
+	char				hook[NG_HOOKSIZ];	/* node hook */
 	char				logname[32];	/* peer logname */
 	struct in_addr			ip;		/* peer ip address */
 	u_int16_t			port;		/* peer port */
@@ -536,7 +536,7 @@
 	struct sockaddr_in sin;
 	const size_t bufsize = 8192;
 	u_int16_t *buf = NULL;
-	char hook[NG_HOOKLEN + 1];
+	char hook[NG_HOOKSIZ];
 	socklen_t sin_len;
 	char namebuf[64];
 	ng_ID_t node_id;
--- ppp/ppp_node.c.orig	2005-01-22 00:02:08.000000000 +0300
+++ ppp/ppp_node.c	2009-06-24 22:00:15.000000000 +0400
@@ -431,7 +431,7 @@
 ppp_node_send_msg(struct ppp_node *node, const char *relpath,
 	u_int32_t cookie, u_int32_t cmd, const void *payload, size_t plen)
 {
-	char path[NG_PATHLEN + 1];
+	char path[NG_PATHSIZ];
 
 	if (relpath == NULL)
 		strlcpy(path, NODE_HOOK, sizeof(path));
@@ -494,7 +494,7 @@
 {
 	struct ppp_node *const node = arg;
 	const size_t max_msglen = 4096;
-	char raddr[NG_PATHLEN + 1];
+	char raddr[NG_PATHSIZ];
 	struct ppp_node_recvmsg *rm;
 	struct ng_mesg *msg;
 	int found = 0;


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->wxs 
Responsible-Changed-By: wxs 
Responsible-Changed-When: Wed Jun 24 18:53:53 UTC 2009 
Responsible-Changed-Why:  
I'll take it. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=136012 
State-Changed-From-To: open->feedback 
State-Changed-By: wxs 
State-Changed-When: Sun Jun 28 13:51:10 UTC 2009 
State-Changed-Why:  
Pending maintainer approval. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=136012 

From: Wesley Shields <wxs@FreeBSD.org>
To: mav@alkar.net
Cc: bug-followup@FreeBSD.org
Subject: Re: ports/136012: libpdel is broken on CURRENT after June,9
Date: Sun, 28 Jun 2009 09:51:03 -0400

 Maintainer of devel/libpdel:
 
 ports/136012 has been submitted. Can you please review this PR and if it
 contains a patch let me know if you approve of it or not.
 
 http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/136012
 
 Thank you.
 
 -- WXS

From: Alexander Motin <mav@FreeBSD.org>
To: Wesley Shields <wxs@FreeBSD.org>
Cc: bug-followup@FreeBSD.org
Subject: Re: ports/136012: libpdel is broken on CURRENT after June,9
Date: Sun, 28 Jun 2009 17:03:54 +0300

 I've sent equal patch to sem@ five days ago, but have no response yet. 
 If somebody can do it faster, please proceed.
 
 -- 
 Alexander Motin
State-Changed-From-To: feedback->closed 
State-Changed-By: sem 
State-Changed-When: Mon Jun 29 15:25:42 UTC 2009 
State-Changed-Why:  
Committed. Thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=136012 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/136012: commit references a PR
Date: Mon, 29 Jun 2009 15:25:27 +0000 (UTC)

 sem         2009-06-29 15:25:18 UTC
 
   FreeBSD ports repository
 
   Added files:
     devel/libpdel/files  patch-len 
   Log:
   - Fix build on 8.0
   
   PR:             ports/136012
   Submitted by:   Boris Lytochkin <lytboris at gmail.com>
   Approved by:    mav (maintainer)
   
   Revision  Changes    Path
   1.1       +80 -0     ports/devel/libpdel/files/patch-len (new)
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
>Unformatted:
