Fix:binding/dbus:Before writing to a file, check if open was successful

Signed-off-by: mvglasow <michael -at- vonglasow.com>

1 files changed, 11 insertions, 0 deletions

diff --git a/navit/binding/dbus/binding_dbus.c b/navit/binding/dbus/binding_dbus.c
index cac1cb339..7fb9f00b2 100644
--- a/navit/binding/dbus/binding_dbus.c
+++ b/navit/binding/dbus/binding_dbus.c
@@ -1345,6 +1345,12 @@ static DBusHandlerResult request_navit_route_export_gpx(DBusConnection *connecti
 
     FILE *fp;
     fp = fopen(filename,"w");
+    if (!fp) {
+        map_rect_destroy(mr);
+        return dbus_error(connection, message, DBUS_ERROR_FAILED,
+                              "could not open file for writing");
+    }
+
     fprintf(fp, "%s", header);
 
     while((item = map_rect_get_item(mr))) {
@@ -1424,6 +1430,11 @@ static DBusHandlerResult request_navit_route_export_geojson(DBusConnection *conn
 
     FILE *fp;
     fp = fopen(filename,"w");
+    if (!fp) {
+        return dbus_error(connection, message, DBUS_ERROR_FAILED,
+                              "could not open file for writing");
+    }
+
     fprintf(fp, "%s", header);
     int is_first=1;
     char * instructions;