On Mon, Mar 25, 2024 at 12:55 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote:
> Vittorio Giovara: > > On Mon, Mar 25, 2024 at 12:38 PM Andreas Rheinhardt < > > andreas.rheinha...@outlook.com> wrote: > > > >> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com> > >> --- > >> libavutil/avstring.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/libavutil/avstring.c b/libavutil/avstring.c > >> index 2071dd36a5..8702fe0455 100644 > >> --- a/libavutil/avstring.c > >> +++ b/libavutil/avstring.c > >> @@ -299,7 +299,7 @@ char *av_append_path_component(const char *path, > const > >> char *component) > >> char *fullpath; > >> > >> if (!path) > >> - return av_strdup(component); > >> + return component ? av_strdup(component) : NULL; > >> if (!component) > >> return av_strdup(path); > >> > > > > isn't this what av_strdup already does? > > It's not documented to do so. It could also decide to treat > av_strdup(NULL) as av_strdup(""). > Ah fair point, but should we not update its documentation instead? -- Vittorio _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".