>>>>> On Thu, 21 Sep 2023 09:33:57 -0700, Paul Eggert said:
> 
> Could you please look at the current sources? That part of 'tar' has 
> changed significantly since 1.35, and I don't see a bug in the latest 
> version. If the bug's still there, please let us know.

The latest version looks correct to me too, but I think it would crash
if the final node in the chain has the _WSNF_JOIN flag.  If this is
supposed to be impossible, then the tests for non-NULL p at lines 571
and 575 are redundant and confusing.

   571    for (p = node; p && (p->flags & _WSNF_JOIN); p = p->next)
   572      {
   573        len += wsnode_len (p);
   574      }
   575    if (p)
   576      len += wsnode_len (p);
   577    end = p;

__Martin

Reply via email to