check string lengths to avoid index out of bound

This commit is contained in:
koranza
2018-05-18 18:25:49 -05:00
committed by GitHub
parent 39988eb1cf
commit aefa6547c3

View File

@@ -139,6 +139,7 @@ proc findExe*(exe: string, followSymlinks: bool = true;
## is added the `ExeExts <#ExeExts>`_ file extensions if it has none.
## If the system supports symlinks it also resolves them until it
## meets the actual file. This behavior can be disabled if desired.
if exe.len == 0: return
template checkCurrentDir() =
for ext in extensions:
result = addFileExt(exe, ext)
@@ -149,8 +150,9 @@ proc findExe*(exe: string, followSymlinks: bool = true;
checkCurrentDir()
let path = string(getEnv("PATH"))
for candidate in split(path, PathSep):
if candidate.len == 0: continue
when defined(windows):
var x = (if candidate[0] == '"' and candidate[^1] == '"':
var x = (if candidate.len > 2 and candidate[0] == '"' and candidate[^1] == '"':
substr(candidate, 1, candidate.len-2) else: candidate) /
exe
else:
@@ -1649,4 +1651,4 @@ proc setLastModificationTime*(file: string, t: times.Time) =
var ft = t.toWinTime.toFILETIME
let res = setFileTime(h, nil, nil, ft.addr)
discard h.closeHandle
if res == 0'i32: raiseOSError(osLastError())
if res == 0'i32: raiseOSError(osLastError())