From root@pr-comm.com  Fri May 30 06:14:11 1997
Received: from cyber2.servtech.com (root@cyber2.servtech.com [199.1.22.12])
          by hub.freebsd.org (8.8.5/8.8.5) with ESMTP id GAA15038
          for <FreeBSD-gnats-submit@freebsd.org>; Fri, 30 May 1997 06:14:10 -0700 (PDT)
Received: from pr-comm.com (root@prcomm.roc.servtech.com [204.181.3.14]) by cyber2.servtech.com (8.8.5/8.8.5) with ESMTP id JAA25545 for <FreeBSD-gnats-submit@freebsd.org>; Fri, 30 May 1997 09:14:07 -0400 (EDT)
Received: (from root@localhost)
	by pr-comm.com (8.8.5/8.8.5) id JAA18495;
	Fri, 30 May 1997 09:10:04 -0400 (EDT)
Message-Id: <199705301310.JAA18495@pr-comm.com>
Date: Fri, 30 May 1997 09:10:04 -0400 (EDT)
From: housley@pr-comm.com
Reply-To: housley@pr-comm.com
To: FreeBSD-gnats-submit@freebsd.org
Subject: Compile of /usr/src/usr.sbin/ppp fails if DEBUG defined
X-Send-Pr-Version: 3.2

>Number:         3718
>Category:       bin
>Synopsis:       Compile of /usr/src/usr.sbin/ppp fails if DEBUG defined
>Confidential:   no
>Severity:       non-critical
>Priority:       high
>Responsible:    brian
>State:          closed
>Quarter:
>Keywords:
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri May 30 06:20:01 PDT 1997
>Closed-Date:    Mon Jun 16 06:58:33 PDT 1997
>Last-Modified:  Mon Jun 16 07:05:58 PDT 1997
>Originator:     James E. Houley
>Release:        FreeBSD 2.2-STABLE i386
>Organization:
PR Communications, Inc.
>Environment:

	

>Description:

	If /usr/src/usr.sbin/ppp is compiled with DEBUG defined
	hdlc.c will fail to compile with an error.

>How-To-Repeat:

	Add CFLAGS += -DDEBUG to the makefile

>Fix:
	

--- hdlc.c	Fri May 30 09:05:21 1997
+++ hdlc.c.orig	Fri May 30 09:04:45 1997
@@ -200,11 +200,11 @@
 void
 DecodePacket(u_short proto, struct mbuf *bp)
 {
-  u_char *cp;
-
 #ifdef DEBUG
   logprintf("proto = %04x\n", proto);
 #endif
+  u_char *cp;
+
   switch (proto) {
   case PROTO_LCP:
     LcpInput(bp);

>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: brian 
State-Changed-When: Mon Jun 16 06:58:33 PDT 1997 
State-Changed-Why:  
All but one #ifdef DEBUGs are now gone (a main() in arp.c 
will be built with DEBUG and expects manual compilation). 
All other DEBUG stuff is now available with "set log +debug". 


Responsible-Changed-From-To: freebsd-bugs->brian 
Responsible-Changed-By: brian 
Responsible-Changed-When: Mon Jun 16 06:58:33 PDT 1997 
Responsible-Changed-Why:  
I fixed it a few days ago. 
>Unformatted:
