On Sun, Dec 01, 2019 at 05:32:04PM +0100, Hendrik Leppkes wrote: > On Sun, Dec 1, 2019 at 3:07 PM <[email protected]> wrote: > > > > From: Limin Wang <[email protected]> > > > > Signed-off-by: Limin Wang <[email protected]> > > --- > > libavformat/utils.c | 7 +++---- > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/libavformat/utils.c b/libavformat/utils.c > > index 8196442..579e6d6 100644 > > --- a/libavformat/utils.c > > +++ b/libavformat/utils.c > > @@ -4854,11 +4854,10 @@ int ff_mkdir_p(const char *path) > > return -1; > > } > > > > - if (!av_strncasecmp(temp, "/", 1) || !av_strncasecmp(temp, "\\", 1)) { > > + if (*temp == '.') > > + pos++; > > + if (*temp == '/' || *temp == '\\') > > pos++; > > - } else if (!av_strncasecmp(temp, "./", 2) || !av_strncasecmp(temp, > > ".\\", 2)) { > > - pos += 2; > > - } > > > > Is the change in logic intentional? It seems problematic to me to > potentially skip any dot, not just dots followed by a (back)slash. > Afterall names can start with dots.
Yes, you're right, I'll update the patch. > > - Hendrik > _______________________________________________ > 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". _______________________________________________ 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".
