On Thu, Aug 8, 2019 at 12:45 PM Christopher Ertl
<christopher.e...@microsoft.com> wrote:
> So I'm proposing to remove the check for the drive letter being alpha in 
> `has_dos_drive_prefix` macro:
>
> #define has_dos_drive_prefix(path) \
>         ( (path)[1] == ':' ? 2 : 0)

Nit: This isn't safe and will access memory beyond end-of-string if
path is zero-length. Perhaps something like this would be better:

    #define has_dos_drive_prefix(path) \
        (*(path) && (path)[1] == ':' ? 2 : 0)

Reply via email to