]> granicus.if.org Git - curl/commitdiff
tool_cb_wrt: fix writing to Windows null device NUL
authorJay Satiro <raysatiro@yahoo.com>
Tue, 26 Mar 2019 07:31:30 +0000 (03:31 -0400)
committerJay Satiro <raysatiro@yahoo.com>
Tue, 26 Mar 2019 07:31:30 +0000 (03:31 -0400)
- Improve console detection.

Prior to this change WriteConsole could be called to write to a handle
that may not be a console, which would cause an error. This issue is
limited to character devices that are not also consoles such as the null
device NUL.

Bug: https://github.com/curl/curl/issues/3175#issuecomment-439068724
Reported-by: Gisle Vanem
src/tool_cb_wrt.c

index 195d6e79c8918f69cc574699129eaa75a37f38e1..1944f16c204fc06a836ba7b319644d3ba2477c35 100644 (file)
@@ -79,6 +79,9 @@ size_t tool_write_cb(char *buffer, size_t sz, size_t nmemb, void *userdata)
   struct OperationConfig *config = outs->config;
   size_t bytes = sz * nmemb;
   bool is_tty = config->global->isatty;
+#ifdef WIN32
+  CONSOLE_SCREEN_BUFFER_INFO console_info;
+#endif
 
   /*
    * Once that libcurl has called back tool_write_cb() the returned value
@@ -156,7 +159,9 @@ size_t tool_write_cb(char *buffer, size_t sz, size_t nmemb, void *userdata)
   }
 
 #ifdef _WIN32
-  if(isatty(fileno(outs->stream))) {
+  if(isatty(fileno(outs->stream)) &&
+     GetConsoleScreenBufferInfo(
+       (HANDLE)_get_osfhandle(fileno(outs->stream)), &console_info)) {
     DWORD in_len = (DWORD)(sz * nmemb);
     wchar_t* wc_buf;
     DWORD wc_len;