summaryrefslogtreecommitdiffstats
path: root/win/tclWinConsole.c
diff options
context:
space:
mode:
authordgp <dgp@users.sourceforge.net>2014-03-21 17:11:32 (GMT)
committerdgp <dgp@users.sourceforge.net>2014-03-21 17:11:32 (GMT)
commit72ae8dccf119510f1b175a9a2243a87069cef308 (patch)
tree1ac746813e875394bef57b70698fcadb2bb2392f /win/tclWinConsole.c
parent148177b382eb972506f96cd8bfb45c6958cea1b6 (diff)
downloadtcl-72ae8dccf119510f1b175a9a2243a87069cef308.zip
tcl-72ae8dccf119510f1b175a9a2243a87069cef308.tar.gz
tcl-72ae8dccf119510f1b175a9a2243a87069cef308.tar.bz2
Added comments raising questions about possible updates to channel
drivers on Windows.
Diffstat (limited to 'win/tclWinConsole.c')
-rw-r--r--win/tclWinConsole.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/win/tclWinConsole.c b/win/tclWinConsole.c
index 0ec22c5..6630083 100644
--- a/win/tclWinConsole.c
+++ b/win/tclWinConsole.c
@@ -756,6 +756,13 @@ ConsoleInputProc(
if (ReadConsoleBytes(infoPtr->handle, (LPVOID) buf, (DWORD) bufSize,
&count) == TRUE) {
+ /*
+ * TODO: This potentially writes beyond the limits specified
+ * by the caller. In practice this is harmless, since all writes
+ * are into ChannelBuffers, and those have padding, but still
+ * ought to remove this, unless some Windows wizard can give
+ * a reason not to.
+ */
buf[count] = '\0';
return count;
}