Vittorio Giovara:
> On Mon, Mar 25, 2024 at 12:38 PM Andreas Rheinhardt <
> [email protected]> wrote:
>
>> Signed-off-by: Andreas Rheinhardt <[email protected]>
>> ---
>> 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("").
- Andreas
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".
