netfilter: fix ABI breakage caused by the netfilter match optimization (fixes #5628)
SVN-Revision: 20552
This commit is contained in:
parent
e4a6acf9b8
commit
5f89a1f096
5 changed files with 115 additions and 0 deletions
|
@ -119,3 +119,26 @@
|
||||||
/* For return from builtin chain */
|
/* For return from builtin chain */
|
||||||
back = get_entry(table_base, private->underflow[hook]);
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
@@ -976,6 +1015,7 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
unsigned int i;
|
||||||
|
const struct ipt_entry_match *m;
|
||||||
|
const struct ipt_entry_target *t;
|
||||||
|
+ u8 flags;
|
||||||
|
|
||||||
|
e = (struct ipt_entry *)(loc_cpu_entry + off);
|
||||||
|
if (copy_to_user(userptr + off
|
||||||
|
@@ -986,6 +1026,14 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
goto free_counters;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ flags = e->ip.flags & ~IPT_F_NO_DEF_MATCH;
|
||||||
|
+ if (copy_to_user(userptr + off
|
||||||
|
+ + offsetof(struct ipt_entry, ip.flags),
|
||||||
|
+ &flags, sizeof(flags)) != 0) {
|
||||||
|
+ ret = -EFAULT;
|
||||||
|
+ goto free_counters;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
for (i = sizeof(struct ipt_entry);
|
||||||
|
i < e->target_offset;
|
||||||
|
i += m->u.match_size) {
|
||||||
|
|
|
@ -119,3 +119,26 @@
|
||||||
/* For return from builtin chain */
|
/* For return from builtin chain */
|
||||||
back = get_entry(table_base, private->underflow[hook]);
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
@@ -978,6 +1017,7 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
unsigned int i;
|
||||||
|
const struct ipt_entry_match *m;
|
||||||
|
const struct ipt_entry_target *t;
|
||||||
|
+ u8 flags;
|
||||||
|
|
||||||
|
e = (struct ipt_entry *)(loc_cpu_entry + off);
|
||||||
|
if (copy_to_user(userptr + off
|
||||||
|
@@ -988,6 +1028,14 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
goto free_counters;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ flags = e->ip.flags & ~IPT_F_NO_DEF_MATCH;
|
||||||
|
+ if (copy_to_user(userptr + off
|
||||||
|
+ + offsetof(struct ipt_entry, ip.flags),
|
||||||
|
+ &flags, sizeof(flags)) != 0) {
|
||||||
|
+ ret = -EFAULT;
|
||||||
|
+ goto free_counters;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
for (i = sizeof(struct ipt_entry);
|
||||||
|
i < e->target_offset;
|
||||||
|
i += m->u.match_size) {
|
||||||
|
|
|
@ -119,3 +119,26 @@
|
||||||
/* For return from builtin chain */
|
/* For return from builtin chain */
|
||||||
back = get_entry(table_base, private->underflow[hook]);
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
@@ -992,6 +1031,7 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
unsigned int i;
|
||||||
|
const struct ipt_entry_match *m;
|
||||||
|
const struct ipt_entry_target *t;
|
||||||
|
+ u8 flags;
|
||||||
|
|
||||||
|
e = (struct ipt_entry *)(loc_cpu_entry + off);
|
||||||
|
if (copy_to_user(userptr + off
|
||||||
|
@@ -1002,6 +1042,14 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
goto free_counters;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ flags = e->ip.flags & ~IPT_F_NO_DEF_MATCH;
|
||||||
|
+ if (copy_to_user(userptr + off
|
||||||
|
+ + offsetof(struct ipt_entry, ip.flags),
|
||||||
|
+ &flags, sizeof(flags)) != 0) {
|
||||||
|
+ ret = -EFAULT;
|
||||||
|
+ goto free_counters;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
for (i = sizeof(struct ipt_entry);
|
||||||
|
i < e->target_offset;
|
||||||
|
i += m->u.match_size) {
|
||||||
|
|
|
@ -119,3 +119,26 @@
|
||||||
/* For return from builtin chain */
|
/* For return from builtin chain */
|
||||||
back = get_entry(table_base, private->underflow[hook]);
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
@@ -992,6 +1031,7 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
unsigned int i;
|
||||||
|
const struct ipt_entry_match *m;
|
||||||
|
const struct ipt_entry_target *t;
|
||||||
|
+ u8 flags;
|
||||||
|
|
||||||
|
e = (struct ipt_entry *)(loc_cpu_entry + off);
|
||||||
|
if (copy_to_user(userptr + off
|
||||||
|
@@ -1002,6 +1042,14 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
goto free_counters;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ flags = e->ip.flags & ~IPT_F_NO_DEF_MATCH;
|
||||||
|
+ if (copy_to_user(userptr + off
|
||||||
|
+ + offsetof(struct ipt_entry, ip.flags),
|
||||||
|
+ &flags, sizeof(flags)) != 0) {
|
||||||
|
+ ret = -EFAULT;
|
||||||
|
+ goto free_counters;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
for (i = sizeof(struct ipt_entry);
|
||||||
|
i < e->target_offset;
|
||||||
|
i += m->u.match_size) {
|
||||||
|
|
|
@ -119,3 +119,26 @@
|
||||||
/* For return from builtin chain */
|
/* For return from builtin chain */
|
||||||
back = get_entry(table_base, private->underflow[hook]);
|
back = get_entry(table_base, private->underflow[hook]);
|
||||||
|
|
||||||
|
@@ -992,6 +1031,7 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
unsigned int i;
|
||||||
|
const struct ipt_entry_match *m;
|
||||||
|
const struct ipt_entry_target *t;
|
||||||
|
+ u8 flags;
|
||||||
|
|
||||||
|
e = (struct ipt_entry *)(loc_cpu_entry + off);
|
||||||
|
if (copy_to_user(userptr + off
|
||||||
|
@@ -1002,6 +1042,14 @@ copy_entries_to_user(unsigned int total_
|
||||||
|
goto free_counters;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ flags = e->ip.flags & ~IPT_F_NO_DEF_MATCH;
|
||||||
|
+ if (copy_to_user(userptr + off
|
||||||
|
+ + offsetof(struct ipt_entry, ip.flags),
|
||||||
|
+ &flags, sizeof(flags)) != 0) {
|
||||||
|
+ ret = -EFAULT;
|
||||||
|
+ goto free_counters;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
for (i = sizeof(struct ipt_entry);
|
||||||
|
i < e->target_offset;
|
||||||
|
i += m->u.match_size) {
|
||||||
|
|
Loading…
Reference in a new issue