Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 12 Mar 2015 17:38:58 -0600
From:      Ian Lepore <ian@freebsd.org>
To:        Navdeep Parhar <np@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r279934 - head/sys/dev/cxgbe
Message-ID:  <1426203538.6265.6.camel@freebsd.org>
In-Reply-To: <5502191D.1010901@FreeBSD.org>
References:  <201503121822.t2CIML1c044098@svn.freebsd.org> <5502191D.1010901@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 2015-03-12 at 15:54 -0700, Navdeep Parhar wrote:
> On 03/12/2015 11:22, Ian Lepore wrote:
> > Author: ian
> > Date: Thu Mar 12 18:22:20 2015
> > New Revision: 279934
> > URL: https://svnweb.freebsd.org/changeset/base/279934
> >
> > Log:
> >    Nullterminate strings returned via sysctl.
> >
> >    PR:		195668
> >
> > Modified:
> >    head/sys/dev/cxgbe/t4_l2t.c
> >    head/sys/dev/cxgbe/t4_main.c
> >
> > Modified: head/sys/dev/cxgbe/t4_l2t.c
> > ==============================================================================
> > --- head/sys/dev/cxgbe/t4_l2t.c	Thu Mar 12 18:09:39 2015	(r279933)
> > +++ head/sys/dev/cxgbe/t4_l2t.c	Thu Mar 12 18:22:20 2015	(r279934)
> > @@ -321,6 +321,7 @@ skip:
> >   		mtx_unlock(&e->lock);
> >   	}
> >
> > +	sbuf_putc(&sb, 0); /* nullterm */
> 
> This breaks the build, please fix.
> 
> Here's the relevant snippet from Jenkins:
> 
> --- t4_l2t.o ---
> <https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/sys/modules/cxgbe/if_cxgbe/../../../dev/cxgbe/t4_l2t.c>:324:12: 
> error: incompatible pointer types passing 'struct sbuf **' to parameter 
> of type 'struct sbuf *'; remove & [-Werror,-Wincompatible-pointer-types]
>          sbuf_putc(&sb, 0); /* nullterm */
>                    ^~~
> <https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/sys/sys/sbuf.h>:77:30: 
> note: passing argument to parameter here
> int              sbuf_putc(struct sbuf *, int);
>                                          ^
> 1 error generated.
> *** [t4_l2t.o] Error code 1
> 

Ooops, sorry about that, I didn't notice the different levels of
indirection in the two files and pasted the same line into both.

-- Ian




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1426203538.6265.6.camel>