| To: | Patrick McHardy <kaber@xxxxxxxxx> |
|---|---|
| Subject: | Re: [Vpn-failover] [RFC] IPSEC failover - Netlink part |
| From: | Ulrich Weber <uweber@xxxxxxxxx> |
| Date: | Tue, 09 Nov 2004 09:55:05 +0100 |
| Cc: | vpn-failover@xxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, ipsec-tools-devel@xxxxxxxxxxxxxxxxxxxxx |
| In-reply-to: | <418F9A70.7040400@xxxxxxxxx> |
| References: | <1099045435.2888.47.camel@xxxxxxxxxxxxxx> <418A3630.1040900@xxxxxxxxx> <418A71DA.2090607@xxxxxxxxx> <418F4AF9.7050605@xxxxxxxxx> <418F9A70.7040400@xxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mozilla Thunderbird 0.8 (X11/20040926) |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
|> |> + NLMSG_LENGTH(sizeof(struct xfrm_usersa_id)),/* UPD SEQ */
|> |>
|> |>
|> | ^^ what about struct xfrm_replay_state ?
|> it's attached as rt attribute, so it should be ok.
|
| But you don't check it's size. You need to include it here or check it
| when you use it.
ok i added the following checkt to ensure the right size.
If i add the size of the xfrm_replay_state struct to the above array, the rt
attribute macros wouldnt work anymore.
Cheers
~ Ulrich
- ----
struct rtattr *rt = xfrma[XFRMA_REPLAY-1];
if( xfrma[XFRMA_REPLAY-1] && (rt->rta_len == (sizeof(struct
xfrm_replay_state) + sizeof(struct rtattr)))) {
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org
iD8DBQFBkIXp22t2oTuElzoRAqsnAJ9QlZuU6wBWtmkzcT8aeb9+AL3nfQCfZUeI
EJh0+e8EOS4/HKiP2Ylu02k=
=roZY
-----END PGP SIGNATURE-----
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH wireless-2.6 0/12] Host AP update, Jeff Garzik |
|---|---|
| Next by Date: | [PATCH 2.6 PKT_SCHED]: Clean up tcf_action_init memory handling, Patrick McHardy |
| Previous by Thread: | Re: [Vpn-failover] [RFC] IPSEC failover - Netlink part, Patrick McHardy |
| Next by Thread: | Re: BUG: dst underflow (again), jamal |
| Indexes: | [Date] [Thread] [Top] [All Lists] |