On Sun, Jul 23, 2023 at 12:55:46PM -0700, Pierre-Anthony Lemieux wrote: > Would this patch be an opportunity to set `cpl->content_title_utf8` to > an empty string at fill_content_title() at libavformat/imf_cpl.c if > xmlNodeListGetString() returns NULL? It could be done as a separate > patch alternatively. ill send a seperate patch for this > > LGTM otherwise. will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB If a bugfix only changes things apparently unrelated to the bug with no further explanation, that is a good sign that the bugfix is wrong.