In NAT forwarding mode, Applications may decrease the size of packets,
and TCP sequences will get smaller, so both of variables will be negetive
values in this case.
Signed-off-by: zhang kai <redacted>
---
include/net/ip_vs.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/net/ip_vs.h b/include/net/ip_vs.h
index 3759167f91f5..de7e75063c7c 100644
--- a/include/net/ip_vs.h
+++ b/include/net/ip_vs.h
@@ -346,8 +346,8 @@ enum ip_vs_sctp_states {
*/
struct ip_vs_seq {
__u32 init_seq; /* Add delta from this seq */
- __u32 delta; /* Delta in sequence numbers */
- __u32 previous_delta; /* Delta in sequence numbers
+ __s32 delta; /* Delta in sequence numbers */
+ __s32 previous_delta; /* Delta in sequence numbers
* before last resized pkt */
};
--
2.17.1
Hello,
Cc list trimmed...
On Tue, 20 Aug 2019, zhang kai wrote:
In NAT forwarding mode, Applications may decrease the size of packets,
and TCP sequences will get smaller, so both of variables will be negetive
values in this case.
As long as nobody cares about their sign, the type should not
matter. You can not solve all signed/unsigned mismatches with such
small patch. Or you are seeing some problem, may be in debug?
quoted hunk
Signed-off-by: zhang kai <redacted>
---
include/net/ip_vs.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/net/ip_vs.h b/include/net/ip_vs.h
index 3759167f91f5..de7e75063c7c 100644
--- a/include/net/ip_vs.h
+++ b/include/net/ip_vs.h
@@ -346,8 +346,8 @@ enum ip_vs_sctp_states {
*/
struct ip_vs_seq {
__u32 init_seq; /* Add delta from this seq */
- __u32 delta; /* Delta in sequence numbers */
- __u32 previous_delta; /* Delta in sequence numbers
+ __s32 delta; /* Delta in sequence numbers */
+ __s32 previous_delta; /* Delta in sequence numbers
* before last resized pkt */
};
--
2.17.1
Regards
--
Julian Anastasov [off-list ref]