[libvirt] [PATCH 2/2] Always open files in binary mode in virFDStreamOpenFileInternal

Claudio Bley cbley at av-test.de
Tue Sep 24 09:57:38 UTC 2013


On win32, using text mode for binary files might result in short
reads since ASCII character 0x1A is interpreted as EOF. Also, it
could lead to problems using the seek functions because of the \r
handling.

Signed-off-by: Claudio Bley <cbley at av-test.de>
---
 src/fdstream.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/fdstream.c b/src/fdstream.c
index 10c7c2a..f7dae39 100644
--- a/src/fdstream.c
+++ b/src/fdstream.c
@@ -593,7 +593,7 @@ virFDStreamOpenFileInternal(virStreamPtr st,
     VIR_DEBUG("st=%p path=%s oflags=%x offset=%llu length=%llu mode=%o",
               st, path, oflags, offset, length, mode);
 
-    oflags |= O_NOCTTY;
+    oflags |= O_NOCTTY | O_BINARY;
 
     if (oflags & O_CREAT)
         fd = open(path, oflags, mode);
-- 
1.8.3.1




More information about the libvir-list mailing list