[tor-commits] [tor/master] src/common/util.c:expand_filename() - Perhaps use GetFullPathName() as a form of input validation on the filename argument.

nickm at torproject.org nickm at torproject.org
Tue Feb 16 16:36:01 UTC 2016


commit 4e19133dccfc4d252e9ed2695f6fe49bb4503ac8
Author: Jeremy <jeremy at sturgix.com>
Date:   Tue Dec 1 15:41:03 2015 -0500

    src/common/util.c:expand_filename() - Perhaps use GetFullPathName() as a form of input validation on the filename argument.
---
 src/common/util.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/common/util.c b/src/common/util.c
index b33c80f..e8044f9 100644
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -2873,6 +2873,9 @@ expand_filename(const char *filename)
 {
   tor_assert(filename);
 #ifdef _WIN32
+  /* Might consider using GetFullPathName() as described here:
+   * http://etutorials.org/Programming/secure+programming/Chapter+3.+Input+Validation/3.7+Validating+Filenames+and+Paths/
+   */
   return tor_strdup(filename);
 #else
   if (*filename == '~') {





More information about the tor-commits mailing list