From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 5 Dec 2016 13:06:52 +0300 From: Dan Carpenter Message-ID: <20161205100652.GA8244@mwanda> References: <20161130195327.GA10274@mwanda> <1800526.pyWDsWJuoq@bentobox> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1800526.pyWDsWJuoq@bentobox> Subject: Re: [B.A.T.M.A.N.] [bug report] batman-adv: make the TT CRC logic VLAN specific List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Sven Eckelmann Cc: b.a.t.m.a.n@lists.open-mesh.org On Fri, Dec 02, 2016 at 05:34:39PM +0100, Sven Eckelmann wrote: > On Mittwoch, 30. November 2016 22:53:27 CET Dan Carpenter wrote: > > Hello Antonio Quartulli, > > > > The patch 7ea7b4a14275: "batman-adv: make the TT CRC logic VLAN > > specific" from Jul 30, 2013, leads to the following static checker > > warning: > [...] > > net/batman-adv/translation-table.c:3313 batadv_send_my_tt_response() > > error: uninitialized symbol 'tt_change'. > > Thank you very much for your excellent report. Antonio seems to be busy > at the moment but Simon already forwarded [1] your suggestion to David. > > Small question - what static checker reported that? I would usually guess > smatch when I see your name but at least it was not reproducible with the > current smatch from http://repo.or.cz/w/smatch.git It's some unreleased Smatch stuff I'm working on. (Very buggy). regards, dan carpenter