[MPlayer-dev-eng] [PATCH 1/2] libmpdemux/mf: Replace sprintf by mp_asprintf

Alexander Strasser eclipse7 at gmx.net
Tue Jun 1 23:20:13 EEST 2021


On 2021-05-12 21:42 +0200, Alexander Strasser wrote:
> On 2021-04-29 18:43 +0200, Alexander Strasser wrote:
> > On 2021-04-28 21:09 +0200, Reimar Döffinger wrote:
> > >
> > > A new variable local to this block would make it much easier to argue memory correctness.
> > > This probably also applies to the other usages of this “fname” variable,
> > > but that’s a somewhat separate issue.
> >
> [...]
> >
> > If you want to review it, I could try to refactor the
> > individual ways to gather the file lists into individual
> > functions. Localizing variables to those functions as
> > much as possible.
>
> Patch attached. Only lightly tested. First wanted to know
> if you like it.

Improved version resubmitted in the new patch set.

  Alexander


More information about the MPlayer-dev-eng mailing list