diff mbox

iproute2: use int instead of long for RTAX_HOPLIMIT compare

Message ID 20100723133910.GC12942@babylon
State Accepted, archived
Delegated to: stephen hemminger
Headers show

Commit Message

Ulrich Weber July 23, 2010, 1:39 p.m. UTC
otherwise "if ((int)val == -1)" will never match on 64 bit systems

Signed-off-by: Ulrich Weber <uweber@astaro.com>
---
 ip/iproute.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

stephen hemminger July 23, 2010, 4:01 p.m. UTC | #1
On Fri, 23 Jul 2010 15:39:10 +0200
Ulrich Weber <uweber@astaro.com> wrote:

> otherwise "if ((int)val == -1)" will never match on 64 bit systems
> 
> Signed-off-by: Ulrich Weber <uweber@astaro.com>
> ---
>  ip/iproute.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/ip/iproute.c b/ip/iproute.c
> index 04b253a..711576e 100644
> --- a/ip/iproute.c
> +++ b/ip/iproute.c
> @@ -494,7 +494,7 @@ int print_route(const struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
>  			val = *(unsigned*)RTA_DATA(mxrta[i]);
>  			switch (i) {
>  			case RTAX_HOPLIMIT:
> -				if ((long)val == -1)
> +				if ((int)val == -1)
>  					val = 0;
>  				/* fall through */
>  			default:

All applied
David Miller July 23, 2010, 7:16 p.m. UTC | #2
From: Stephen Hemminger <shemminger@vyatta.com>
Date: Fri, 23 Jul 2010 09:01:39 -0700

> All applied

Please update patchwork, and in the future please do so before emailing
the submitter that the patch is applied.

Patches assigned to you tend to just pile up, defeating the whole purpose
of using patchwork.

If you really don't want to stay on top of this and actually use the
tool, just let me know and I'll just toss all the patches that come
into the netdev group for iproute2.

This is about the 4th or 5th time I've brought this issue up with you.

I know you might find this hard to believe, but it will be less work
for you if the first thing you do as iproute2 maintainer every day is
open up the patchwork web page and scan the incoming patches and
process them, instead of hitting them via the mailing list first. :-)

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/ip/iproute.c b/ip/iproute.c
index 04b253a..711576e 100644
--- a/ip/iproute.c
+++ b/ip/iproute.c
@@ -494,7 +494,7 @@  int print_route(const struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
 			val = *(unsigned*)RTA_DATA(mxrta[i]);
 			switch (i) {
 			case RTAX_HOPLIMIT:
-				if ((long)val == -1)
+				if ((int)val == -1)
 					val = 0;
 				/* fall through */
 			default: