diff options
author | David Sommerseth <dazo@users.sourceforge.net> | 2010-04-17 21:03:06 +0200 |
---|---|---|
committer | David Sommerseth <dazo@users.sourceforge.net> | 2010-04-29 20:22:29 +0200 |
commit | 90919b019c9db272113c885945014b098a7f987b (patch) | |
tree | 663e469fd68f249d63c0233cb76b9e0c437ba827 /ssl.c | |
parent | Removed no longer needed delete_file() call (diff) | |
download | openvpn-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>
Diffstat (limited to '')
-rw-r--r-- | ssl.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -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); |