From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [PATCH] fix nf_conntrack_netlink expectation dumping/event notification Date: Fri, 13 Jan 2006 09:58:15 +0100 Message-ID: <43C76BA7.9080204@trash.net> References: <43C70536.7000608@netfilter.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Netfilter Development Mailinglist , Yasuyuki Kozakai Return-path: To: Pablo Neira Ayuso In-Reply-To: <43C70536.7000608@netfilter.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: netfilter-devel-bounces@lists.netfilter.org Errors-To: netfilter-devel-bounces@lists.netfilter.org List-Id: netfilter-devel.vger.kernel.org Pablo Neira Ayuso wrote: > Hi Yasuyuki, > > Currently we get an oops with nf_conntrack_netlink + nf_conntrack_ftp > because l3num is set to 0xFFFF for the expectation mask. At first sight, > this is correct because l3num is u_int16_t, but the size of the layer-3 > array of protocol handlers is AF_MAX (32). > > I could add some checking to verify that l3num is less than 32 in > nf_conntrack_find_l3proto, but such checking is only required for > nf_conntrack_ftp and further application helpers. AFAICS, this is the > cleanest way to fix this problem. Any other suggestion? What is the exact cause for the crash? I looked over the code, but couldn't spot it.