From 1bd962d9f9037ed20fdf588d3b91ce36567f4867 Mon Sep 17 00:00:00 2001
From: anonimal <anonimal@getmonero.org>
Date: Fri, 6 Sep 2019 23:11:37 +0000
Subject: [PATCH] wallet2: resolve CID 203918 null pointer deference
 (NULL_RETURNS)

---
 src/wallet/wallet2.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/wallet/wallet2.cpp b/src/wallet/wallet2.cpp
index b99fc12e2..009a0441f 100644
--- a/src/wallet/wallet2.cpp
+++ b/src/wallet/wallet2.cpp
@@ -13143,6 +13143,12 @@ bool wallet2::save_to_file(const std::string& path_to_file, const std::string& r
   }
 
   FILE *fp = fopen(path_to_file.c_str(), "w+");
+  if (!fp)
+  {
+    MERROR("Failed to open wallet file for writing: " << path_to_file << ": " << strerror(errno));
+    return false;
+  }
+
   // Save the result b/c we need to close the fp before returning success/failure.
   int write_result = PEM_write(fp, ASCII_OUTPUT_MAGIC.c_str(), "", (const unsigned char *) raw.c_str(), raw.length());
   fclose(fp);