From owner-freebsd-wireless@FreeBSD.ORG Thu Jul 7 12:50:50 2011 Return-Path: Delivered-To: freebsd-wireless@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id AC53E106566B; Thu, 7 Jul 2011 12:50:50 +0000 (UTC) (envelope-from hselasky@c2i.net) Received: from swip.net (mailfe07.c2i.net [212.247.154.194]) by mx1.freebsd.org (Postfix) with ESMTP id 09EC48FC1E; Thu, 7 Jul 2011 12:50:49 +0000 (UTC) X-Cloudmark-Score: 0.000000 [] X-Cloudmark-Analysis: v=1.1 cv=SJA03SmFjIOBaRUqQpAxZTuC3Mg0Qr8luS0qYjfHCY4= c=1 sm=1 a=SvYTsOw2Z4kA:10 a=8GCaWQNJa_IA:10 a=WQU8e4WWZSUA:10 a=8nJEP1OIZ-IA:10 a=CL8lFSKtTFcA:10 a=i9M/sDlu2rpZ9XS819oYzg==:17 a=zO0weGsyaNLyVeU5gJ4A:9 a=wPNLvfGTeEIA:10 a=i9M/sDlu2rpZ9XS819oYzg==:117 Received: from [188.126.198.129] (account mc467741@c2i.net HELO laptop002.hselasky.homeunix.org) by mailfe07.swip.net (CommuniGate Pro SMTP 5.2.19) with ESMTPA id 149805704; Thu, 07 Jul 2011 14:50:48 +0200 From: Hans Petter Selasky To: freebsd-current@freebsd.org Date: Thu, 7 Jul 2011 14:48:58 +0200 User-Agent: KMail/1.13.5 (FreeBSD/8.2-STABLE; KDE/4.4.5; amd64; ; ) References: <1309237117.88943.YahooMailNeo@web39307.mail.mud.yahoo.com> <201107061719.38445.hselasky@c2i.net> In-Reply-To: X-Face: *nPdTl_}RuAI6^PVpA02T?$%Xa^>@hE0uyUIoiha$pC:9TVgl.Oq,NwSZ4V" =?iso-8859-1?q?=7CLR=2E+tj=7Dg5=0A=09=25V?=,x^qOs~mnU3]Gn; cQLv&.N>TrxmSFf+p6(30a/{)KUU!s}w\IhQBj}[g}bj0I3^glmC( =?iso-8859-1?q?=0A=09=3AAuzV9=3A=2EhESm-x4h240C=609=3Dw?= MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201107071448.58436.hselasky@c2i.net> Cc: PseudoCylon , "freebsd-wireless@freebsd.org" Subject: Re: [CFT] Sierra Wireless HSPA+ USB modem X-BeenThere: freebsd-wireless@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussions of 802.11 stack, tools device driver development." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 Jul 2011 12:50:50 -0000 On Thursday 07 July 2011 14:43:22 PseudoCylon wrote: > The compiler complained about uninitialized int > if_usie.c: 1484 > - uint8_t pad; > + uint8_t pad = 0; I changed it so that pad is set in both cases: pad = (hip->id & USIE_HIP_PAD) ? 1 : 0; if ((hip->id & USIE_HIP_MASK) == USIE_HIP_CNS2H) { cns = (struct usie_cns *)(((uint8_t *)(hip + 1)) + pad); --HPS