diff options
| author | jan.nijtmans <nijtmans@users.sourceforge.net> | 2013-09-04 07:22:45 (GMT) |
|---|---|---|
| committer | jan.nijtmans <nijtmans@users.sourceforge.net> | 2013-09-04 07:22:45 (GMT) |
| commit | 5e237c28521e9a2146369913d30a10b2712b9fc2 (patch) | |
| tree | e80d5f727d70e081a7dc7285b869d914aa1eb186 /unix | |
| parent | 5441277b9a31d73e4d91383584882c9303e5c758 (diff) | |
| download | tcl-5e237c28521e9a2146369913d30a10b2712b9fc2.zip tcl-5e237c28521e9a2146369913d30a10b2712b9fc2.tar.gz tcl-5e237c28521e9a2146369913d30a10b2712b9fc2.tar.bz2 | |
Fix 3 trivial (possible) errors, discovered by covertity.com
Diffstat (limited to 'unix')
| -rw-r--r-- | unix/tclUnixFCmd.c | 4 | ||||
| -rw-r--r-- | unix/tclUnixSock.c | 2 |
2 files changed, 4 insertions, 2 deletions
diff --git a/unix/tclUnixFCmd.c b/unix/tclUnixFCmd.c index e27f78f..e270b6a 100644 --- a/unix/tclUnixFCmd.c +++ b/unix/tclUnixFCmd.c @@ -462,10 +462,10 @@ DoCopyFile( switch ((int) (statBufPtr->st_mode & S_IFMT)) { #ifndef DJGPP case S_IFLNK: { - char linkBuf[MAXPATHLEN]; + char linkBuf[MAXPATHLEN+1]; int length; - length = readlink(src, linkBuf, sizeof(linkBuf)); + length = readlink(src, linkBuf, MAXPATHLEN); /* INTL: Native. */ if (length == -1) { return TCL_ERROR; diff --git a/unix/tclUnixSock.c b/unix/tclUnixSock.c index 9c3d7eb..a6360c2 100644 --- a/unix/tclUnixSock.c +++ b/unix/tclUnixSock.c @@ -1358,6 +1358,7 @@ Tcl_OpenTcpServer( my_errno = errno; } close(sock); + sock = -1; continue; } if (port == 0 && chosenport == 0) { @@ -1380,6 +1381,7 @@ Tcl_OpenTcpServer( my_errno = errno; } close(sock); + sock = -1; continue; } if (statePtr == NULL) { |