Potential miss usage of strchr VS strrchr in Wine (was: [PATCH] wscript: Get extension name by strrchrW instead of strchrW.)

Qian Hong qhong at codeweavers.com
Wed Nov 13 04:00:04 CST 2013


Hello,

While working on a vbscript bug affects a real world app, I found a
potential miss usage of strchr/strrch in Wine. Here is a unfiltered
list:

$ git grep "= strchr" | grep ext | grep -E "'\.'"

dlls/gdi32/freetype.c:    ext = strchr( last_part, '.' );
dlls/msi/action.c:    extension = strchrW( filename, '.' );
dlls/wineps.drv/type1afm.c:     CHAR *file_extension =
strchr(dent->d_name, '.');
tools/winebuild/utils.c:    if (!(ext = strchr( basename, '.' ))) ext
= prefix + strlen(prefix);

After basic reviewing, I think all the above codes are potential miss
usage, but there is a risk to touch these codes. I don't know any real
word app affected by these codes yet, so I have no strong motivation
to touch them. Anyone worked on related fields has interesting to
review deeper?

Thanks!

On Tue, Nov 12, 2013 at 4:19 PM, Qian Hong <qhong at codeweavers.com> wrote:
> ---
>  programs/wscript/main.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
>
>
>
>



-- 
Regards,
Qian Hong

-
http://www.codeweavers.com



More information about the wine-devel mailing list