All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: David Miller <davem@davemloft.net>
Cc: arnaud@andesi.org, netfilter-devel@lists.netfilter.org
Subject: Re: libipt_multiport: getsockopt failed strangely: Invalid argument
Date: Sat, 02 Dec 2006 00:51:20 +0100	[thread overview]
Message-ID: <4570BFF8.4060107@trash.net> (raw)
In-Reply-To: <20061201.152406.32722891.davem@davemloft.net>

[-- Attachment #1: Type: text/plain, Size: 539 bytes --]

David Miller wrote:
> From: Patrick McHardy <kaber@trash.net>
> Date: Fri, 01 Dec 2006 17:24:39 +0100
> 
> 
>>I *think* this should be fixed in 2.6.19. The 32 bit compat code
>>didn't implement the SO_GET_REVISION_MATCH/TARGET options and
>>returned an error.
> 
> 
> Agreed.
> 
> Once I have the fix I'll push it around to the -stable
> branches for you Patrick.
> 

Its already in 2.6.19, but I didn't expect it to really fix
something. I've attached it again, it applies all the way
back to 2.6.17 when the compat code was introduced.


[-- Attachment #2: x --]
[-- Type: text/plain, Size: 1265 bytes --]

[NETFILTER]: ip_tables: revision support for compat code

Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>

---
commit 79030ed07de673e8451a03aecb9ada9f4d75d491
tree 4ba8bd843c8bc95db0ea6877880b73d06da620e5
parent bec71b162747708d4b45b0cd399b484f52f2901a
author Patrick McHardy <kaber@trash.net> Wed, 20 Sep 2006 12:05:08 -0700
committer David S. Miller <davem@sunset.davemloft.net> Fri, 22 Sep 2006 15:20:00 -0700

 net/ipv4/netfilter/ip_tables.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/net/ipv4/netfilter/ip_tables.c b/net/ipv4/netfilter/ip_tables.c
index 3d5d4a4..673581d 100644
--- a/net/ipv4/netfilter/ip_tables.c
+++ b/net/ipv4/netfilter/ip_tables.c
@@ -1994,6 +1994,8 @@ compat_get_entries(struct compat_ipt_get
 	return ret;
 }
 
+static int do_ipt_get_ctl(struct sock *, int, void __user *, int *);
+
 static int
 compat_do_ipt_get_ctl(struct sock *sk, int cmd, void __user *user, int *len)
 {
@@ -2007,8 +2009,7 @@ compat_do_ipt_get_ctl(struct sock *sk, i
 		ret = compat_get_entries(user, len);
 		break;
 	default:
-		duprintf("compat_do_ipt_get_ctl: unknown request %i\n", cmd);
-		ret = -EINVAL;
+		ret = do_ipt_get_ctl(sk, cmd, user, len);
 	}
 	return ret;
 }

  reply	other threads:[~2006-12-01 23:51 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-01 12:00 libipt_multiport: getsockopt failed strangely: Invalid argument Arnaud Fontaine
2006-12-01 16:24 ` Patrick McHardy
2006-12-01 23:24   ` David Miller
2006-12-01 23:51     ` Patrick McHardy [this message]
2006-12-02  4:15       ` David Miller
  -- strict thread matches above, loose matches on Subject: below --
2006-11-27 18:54 Arnaud Fontaine

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=4570BFF8.4060107@trash.net \
    --to=kaber@trash.net \
    --cc=arnaud@andesi.org \
    --cc=davem@davemloft.net \
    --cc=netfilter-devel@lists.netfilter.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.