Re: [PATCH] trace2: tolerate failed timestamp formatting
Derrick Stolee <stolee@gmail.com> writes:
Show 14 quoted lines
> * Taylor's hunch that the memory allocation is more likely at fault
> is seeming more and more correct. When we fixed this issue, other
> issues around memory allocation came to light.
>
> * For that reason, I'll rework this patch to point at the allocation
> as the likely reason the parsing fails. Avoiding a die() in the
> tracing code is still critical.
>
> * Thus, I'll also replace the xstrdup() in the trace code to avoid a
> die() due to allocation problems.
>
> * I will take a deeper look at this wrapper change and how it might
> be done in a careful way, as Taylor says his patch was an example
> only and not the "right" way to do it.