Hi Dan, > > + /* copy the ie's value into ht_capb*/ > > + memcpy((u8 *)&sta_ptr->tdls_cap.ht_capb, pos + 2, > ^^^^^^^ > > I don't understand why we changed "pos" to "pos + 2". Presumably there is > a reason, but it needs to explained in the commit message. I think, we were doing wrong in the original code. We are supposed to use 'pos + 2' itself, instead of just 'pos'. This is because, 'pos' is pointing to 'ieee_types_header', followed by the actual data and the destination do not start with (i.e. it do not contain) 'ieee_types_header'(ex: 'sta_ptr->tdls_cap.ht_oper'). Also, there are few places were the destination starts with 'ieee_types_header'(ex: 'sta_ptr->tdls_cap.extcap'), which need just 'pos'. Regards, Ganapathi
