diff options
| author | Padma, Santhosh Kumar <skpadma@qti.qualcomm.com> | 2016-02-18 12:57:30 +0530 |
|---|---|---|
| committer | Anjaneedevi Kapparapu <akappa@codeaurora.org> | 2016-02-22 12:19:15 +0530 |
| commit | bb42b08f96d8e1230cbf37113d521ddf56b46309 (patch) | |
| tree | d3c5fdb16801fa62f22ecb96c90727f042d9adb3 | |
| parent | 7e626fb48f96a6d88fda068baf504d4789d577e2 (diff) | |
qcacld-2.0: Remove NLMSG_ALIGN in pkt_stats_fill_headers
NLMSG_ALIGN adds 1/2/3 bytes if length of data is not multiple of 4.
This can cause length mismatch between nl_payload_len (without header
length) and skb->len. This can cause mismatch in length vs message
available on the socket check in nl and can result in NL discarding
the message. Fix this by removing NLMSG_ALIGN to make lengths same.
Change-Id: I5e513a448a13450b590960ac49462af27e2eb385
CRs-Fixed: 978511
| -rw-r--r-- | CORE/SVC/src/logging/wlan_logging_sock_svc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/CORE/SVC/src/logging/wlan_logging_sock_svc.c b/CORE/SVC/src/logging/wlan_logging_sock_svc.c index e5e3268d4085..fb51ec2d8386 100644 --- a/CORE/SVC/src/logging/wlan_logging_sock_svc.c +++ b/CORE/SVC/src/logging/wlan_logging_sock_svc.c @@ -485,7 +485,7 @@ static int pkt_stats_fill_headers(struct sk_buff *skb) extra_header_len = sizeof(msg_header.radio) + sizeof(tAniHdr) + sizeof(struct nlmsghdr); - nl_payload_len = NLMSG_ALIGN(extra_header_len + skb->len); + nl_payload_len = extra_header_len + skb->len; msg_header.nlh.nlmsg_type = ANI_NL_MSG_PUMAC; msg_header.nlh.nlmsg_len = nl_payload_len; |