aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Sommerseth <dazo@users.sourceforge.net>2010-04-17 21:03:06 +0200
committerDavid Sommerseth <dazo@users.sourceforge.net>2010-04-29 20:22:29 +0200
commit90919b019c9db272113c885945014b098a7f987b (patch)
tree663e469fd68f249d63c0233cb76b9e0c437ba827
parentRemoved no longer needed delete_file() call (diff)
downloadopenvpn-90919b019c9db272113c885945014b098a7f987b.tar.xz
Fixed potential NULL pointer issue
If create_temp_file() returns NULL, this strlen() check would cause a SEGV. Signed-off-by: David Sommerseth <dazo@users.sourceforge.net> Acked-by: Gert Doering <gert@greenie.muc.de>
-rw-r--r--ssl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/ssl.c b/ssl.c
index 3047719..44a1bb2 100644
--- a/ssl.c
+++ b/ssl.c
@@ -3194,7 +3194,7 @@ verify_user_pass_script (struct tls_session *session, const struct user_pass *up
}
done:
- if (strlen (tmp_file) > 0)
+ if (tmp_file && strlen (tmp_file) > 0)
delete_file (tmp_file);
argv_reset (&argv);