From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Erich E. Hoover" Subject: Re: [PATCH v3 1/2] Implement IP_UNICAST_IF socket option. Date: Mon, 6 Feb 2012 12:51:24 -0700 Message-ID: References: <62162DF05402B341B3DB59932A1FA992B5B5CA0154@EUSAACMS0702.eamcs.ericsson.se> <62162DF05402B341B3DB59932A1FA992B5B5CA01B7@EUSAACMS0702.eamcs.ericsson.se> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Linux Netdev To: Shawn Lu Return-path: Received: from izzard.Mines.EDU ([138.67.132.197]:48547 "EHLO izzard.mines.edu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754548Ab2BFTv0 convert rfc822-to-8bit (ORCPT ); Mon, 6 Feb 2012 14:51:26 -0500 Received: from mail-vx0-f174.google.com (mail-vx0-f174.google.com [209.85.220.174]) (authenticated bits=0) by izzard.mines.edu (8.13.8/8.13.8) with ESMTP id q16JpOGT008572 (version=TLSv1/SSLv3 cipher=RC4-SHA bits=128 verify=FAIL) for ; Mon, 6 Feb 2012 12:51:25 -0700 Received: by vcge1 with SMTP id e1so4146131vcg.19 for ; Mon, 06 Feb 2012 11:51:24 -0800 (PST) In-Reply-To: <62162DF05402B341B3DB59932A1FA992B5B5CA01B7@EUSAACMS0702.eamcs.ericsson.se> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, Feb 6, 2012 at 12:40 PM, Shawn Lu wrote= : >... > What I mean is replace > =A0if (sk->sk_bound_dev_if && ifindex !=3D sk->sk_bound_dev_if) > With > =A0if (sk->sk_bound_dev_if) Are you sure that that's appropriate? I choose to do it this way since IP_MULTICAST_IF does the exact same check. Erich Hoover ehoover@mines.edu