[Pkg-wmaker-commits] [wmforecast] 35/63: check return value of mkdir (Coverity 65085)

Doug Torrance dtorrance-guest at moszumanska.debian.org
Mon Aug 17 11:20:35 UTC 2015


This is an automated email from the git hooks/post-receive script.

dtorrance-guest pushed a commit to branch master
in repository wmforecast.

commit 7ea276fb4db0aae0738aee8bfb01124be5614f33
Author: Doug Torrance <dtorrance at monmouthcollege.edu>
Date:   Fri May 30 23:20:07 2014 -0600

    check return value of mkdir (Coverity 65085)
---
 ChangeLog    |  2 ++
 wmforecast.c | 11 ++++++++---
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index e2c2ee7..335a735 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,8 @@
 2014-05-30  Doug Torrance <dtorrance at monmouthcollege.edu>
 	* wmforecast.c (getWeather): fix missing return statements (Coverity
 	  65086)
+	* wmforecast.c (savePreferences): check return value of mkdir (Coverity
+	  65085)
 
 2014-05-11  Doug Torrance <dtorrance at monmouthcollege.edu>
 	* Makefile.am: use dist_man_MANS instead of adding man_MANS to EXTRA_DIST
diff --git a/wmforecast.c b/wmforecast.c
index 648469c..a609e06 100755
--- a/wmforecast.c
+++ b/wmforecast.c
@@ -20,6 +20,7 @@
 #endif
 
 #include <curl/curl.h>
+#include <errno.h>
 #include <getopt.h>
 #include <libgen.h>
 #include <libxml/tree.h>
@@ -690,6 +691,7 @@ static void savePreferences(WMWidget *widget, void *data)
 	char *prefsString;
 	Dockapp *d = (Dockapp *)data;
 	FILE *file;
+	int md;
 	WMPropList *prefsPL;
 	WMPropList *key;
 	WMPropList *object;
@@ -733,12 +735,15 @@ static void savePreferences(WMWidget *widget, void *data)
 	prefsString = WMGetPropListDescription(prefsPL, True);
 
 	dir = getPreferencesDirectory();
-	mkdir(dir, 0777);
+	md = mkdir(dir, 0777);
+	if (md < 0) {
+		printf("Error making directory %s: %s\n", dir,
+		       strerror(errno));
+	}
 
 	filename = getPreferencesFilename();
 	file = fopen(filename, "w");
-	if (file)
-	{
+	if (file) {
 		fputs(prefsString, file);
 		fclose(file);
 	}

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-wmaker/wmforecast.git



More information about the Pkg-wmaker-commits mailing list