netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Joe Harvell <joe.harvell@tekcomms.com>
To: <netdev@vger.kernel.org>
Cc: Stephen Hemminger <shemming@brocade.com>,
	Vadim Kochan <vadim4j@gmail.com>
Subject: [PATCH] iproute2: fix broken get_prefix_1
Date: Sat, 21 Mar 2015 15:07:31 -0500	[thread overview]
Message-ID: <550DCF83.6000403@tekcomms.com> (raw)

Fixes bug that causes a basic 'ip addr add' command to fail address due 
to the address prefix is incorrectly determined to be invalid.

I assume master must have some new change because this fix was needed 
for a basic 'ip addr add 10.0.3.1/24 dev dumbo label foo' command I 
pased in.  In this case, 'family' passed into get_addr_1 two lines above 
is zero, causing get_addr_1 to detect the family from the address and 
populate the result in the family field in dst.  But then instead of 
passing in the result, family (still 0) is passed in to af_bit_len. 
Without my change, the above command complains that 10.0.3.1/24 is not 
an address prefix.  With the change it works fine as expected.

The following changes since commit 4612d04d6b8f07274bd5d0688f717ccc189499ad:

   tc class: Show class names from file (2015-03-15 12:27:40 -0700)

are available in the git repository at:

   git@github.com:jharvell/iproute2.git fix-broken-get_prefix_1

for you to fetch changes up to d24d5cb9ad9b09ceb37aa8ffe8c1160c14f713ef:

   Signed-off-by: Joe Harvell <joe.harvell@tekcomms.com> (2015-03-21 
15:03:32 -0500)

----------------------------------------------------------------
Joe Harvell (2):
       Fixing obvious error of passing in the wrong variable for the 
family parameter     of af_bit_len.
       Signed-off-by: Joe Harvell <joe.harvell@tekcomms.com>

  lib/utils.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/utils.c b/lib/utils.c
index 9cda268..0d08a86 100644
--- a/lib/utils.c
+++ b/lib/utils.c
@@ -477,7 +477,7 @@ int get_prefix_1(inet_prefix *dst, char *arg, int 
family)

         err = get_addr_1(dst, arg, family);
         if (err == 0) {
-               dst->bitlen = af_bit_len(family);
+               dst->bitlen = af_bit_len(dst->family);

                 if (slash) {
                         if (get_netmask(&plen, slash+1, 0)

             reply	other threads:[~2015-03-21 20:08 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-21 20:07 Joe Harvell [this message]
  -- strict thread matches above, loose matches on Subject: below --
2015-03-21 17:46 [PATCH] iproute2: fix broken get_prefix_1 Joe Harvell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=550DCF83.6000403@tekcomms.com \
    --to=joe.harvell@tekcomms.com \
    --cc=netdev@vger.kernel.org \
    --cc=shemming@brocade.com \
    --cc=vadim4j@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).