FST: Avoid using pointer to mgmt->u.action.u.fst_action
authorJouni Malinen <jouni@qca.qualcomm.com>
Fri, 17 Jul 2015 17:37:15 +0000 (20:37 +0300)
committerJouni Malinen <j@w1.fi>
Fri, 17 Jul 2015 17:37:15 +0000 (20:37 +0300)
commit6462e7387d558fa4a95e98e61d5d5cd35a2d7690
tree82ece4f2dff6445fc0576eb9cfb4eacf67f74195
parent85b563f7a4907e20d3dd91dd01cecbd71ea4bade
FST: Avoid using pointer to mgmt->u.action.u.fst_action

Typecasting &mgmt->u.action.u.fst_action to a struct pointer for various
FST Action frame payloads seemed to be triggering static analyzer
warnings about bounds checking since sizeof(mgmt->u.action.u.fst_action)
== 1 even though that is really a variable length structure. Try to
avoid this by calculating the pointer for the beginning of the frame
instead of variable length struct. (CID 125642)

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
src/fst/fst_session.c