From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Marek Lindner Date: Mon, 23 Jan 2012 00:26:08 +0800 References: <1327238153-8303-1-git-send-email-siwu@hrz.tu-chemnitz.de> In-Reply-To: <1327238153-8303-1-git-send-email-siwu@hrz.tu-chemnitz.de> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Message-Id: <201201230026.09036.lindner_marek@yahoo.de> Subject: Re: [B.A.T.M.A.N.] [PATCH] batman-adv: fix initialised flag for tt request/responses Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: The list for a Better Approach To Mobile Ad-hoc Networking On Sunday, January 22, 2012 21:15:53 Simon Wunderlich wrote: > When receiving a tt_response, we first delete the old entries and > set the initialised flag to false, then add the new received entries, > but forgot to set the initialised flag to true again. This may lead to > endless loops of requests/responses, as a request is always issued > if the initialised flag is not set. > > Fix this by moving initialised = true from tt_update_changes() to > _tt_update_changes(). This is a regression introduced by 8ef201e. Can we add this in the commit message somewhere ? As a reminder for me to squash these 2 patches when preparing the patches for David. Thanks, Marek