From bfe783f167d4d88123117a460cde652707bdb2ef Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 27 Mar 2014 23:53:03 -0400 Subject: Make dump_desc() use binary mode Otherwise, it could mung the thing that came over the net on windows, which would defeat the purpose of recording the unparseable thing. Fixes bug 11342; bugfix on 0.2.2.1-alpha. --- changes/bug11342 | 4 ++++ src/or/routerparse.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) create mode 100644 changes/bug11342 diff --git a/changes/bug11342 b/changes/bug11342 new file mode 100644 index 0000000000..a52a5515e7 --- /dev/null +++ b/changes/bug11342 @@ -0,0 +1,4 @@ + o Minor bugfixes: + - When dumping a malformed directory object to disk, save it in binary + mode on windows, not text mode. Fixes bug 11342; bugfix on + 0.2.2.1-alpha. diff --git a/src/or/routerparse.c b/src/or/routerparse.c index 3aa4bdf8a5..d52a98259f 100644 --- a/src/or/routerparse.c +++ b/src/or/routerparse.c @@ -598,7 +598,7 @@ dump_desc(const char *desc, const char *type) char *content = tor_malloc_zero(filelen); tor_snprintf(content, filelen, "Unable to parse descriptor of type " "%s:\n%s", type, desc); - write_str_to_file(debugfile, content, 0); + write_str_to_file(debugfile, content, 1); log_info(LD_DIR, "Unable to parse descriptor of type %s. See file " "unparseable-desc in data directory for details.", type); tor_free(content); -- cgit v1.2.3-54-g00ecf