It seems that the first u32 after the header for some stats are used by
something not documented.

The stats struct starts after it.

Signed-off-by: Mauro Carvalho Chehab <[email protected]>
---
 drivers/media/common/siano/smsdvb.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/media/common/siano/smsdvb.c 
b/drivers/media/common/siano/smsdvb.c
index a5f5272..70ea3e9 100644
--- a/drivers/media/common/siano/smsdvb.c
+++ b/drivers/media/common/siano/smsdvb.c
@@ -724,7 +724,8 @@ static int smsdvb_onresponse(void *context, struct 
smscore_buffer_t *cb)
                        smsdvb_update_isdbt_stats(client, p);
                        break;
                default:
-                       smsdvb_update_dvb_stats(client, p);
+                       /* Skip SmsMsgStatisticsInfo_ST:RequestResult field */
+                       smsdvb_update_dvb_stats(client, p + sizeof(u32));
                }
 
                is_status_update = true;
@@ -732,7 +733,8 @@ static int smsdvb_onresponse(void *context, struct 
smscore_buffer_t *cb)
 
        /* Only for ISDB-T */
        case MSG_SMS_GET_STATISTICS_EX_RES:
-               smsdvb_update_isdbt_stats_ex(client, p);
+               /* Skip SmsMsgStatisticsInfo_ST:RequestResult field? */
+               smsdvb_update_isdbt_stats_ex(client, p + sizeof(u32));
                is_status_update = true;
                break;
        default:
-- 
1.8.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to