Hi Magesh, 

That code is pretty old, could you try with master latest? There have been at 
least a couple of fixes to the bt since then (see [1] and [2]). And, as per 
assert that follows, the bts should be valid and therefore the bug is most 
probably in a different place.

On a separate note, the byte tracker is disabled by default, could you provide 
more details with respect to your use case? 

Regards, 
Florin

[1] https://gerrit.fd.io/r/c/vpp/+/29623
[2] https://gerrit.fd.io/r/c/vpp/+/29428 

> On Nov 22, 2021, at 6:49 AM, magesh parthasarathy <mageshmp2...@gmail.com> 
> wrote:
> 
> We got a crash at the following lines of code in src/vnet/tcp/tcp_bt.c . The 
> code base is 
> vpp_1908 branch.
> 
>   354 /* Find original tx sample and cache flags in case the sample
>   355  * is freed or the pool moves */
>   356  bts = bt_lookup_seq (bt, start);
>   357  bts_flags = bts->flags;
> 
> when bt_lookup_seq returns NULL bts=NULL and accessing bts->flags causes the 
> system to crash.
> There should be a NULL condition checking statement before line 357.
> 
> I checked the master code base and even there the code still remains the same.
>   359  /* Find original tx sample and cache flags in case the sample
>   360    * is freed or the pool moves */
>   361     bts = bt_lookup_seq (bt, start);
>   362     bts_flags = bts->flags;
> 
> Can there be a fix suggested for this bug ?
> 
> Regards
> Magesh
> Truminds software
> 
> 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#20530): https://lists.fd.io/g/vpp-dev/message/20530
Mute This Topic: https://lists.fd.io/mt/87236362/21656
Group Owner: vpp-dev+ow...@lists.fd.io
Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to