From 6d55c2c490ef3c00b18548fbafbdc534a721e8e6 Mon Sep 17 00:00:00 2001
From: John Scipione <jscipione@gmail.com>
Date: Tue, 25 Feb 2014 13:35:08 -0500
Subject: [PATCH] ScreenSaver: check function return values for success

---
 src/preferences/screensaver/ScreenSaverWindow.cpp | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/preferences/screensaver/ScreenSaverWindow.cpp b/src/preferences/screensaver/ScreenSaverWindow.cpp
index 58cbb45234..b17cb246d5 100644
--- a/src/preferences/screensaver/ScreenSaverWindow.cpp
+++ b/src/preferences/screensaver/ScreenSaverWindow.cpp
@@ -702,8 +702,10 @@ ModulesView::PopulateScreenSaverList()
 
 	for (uint32 i = 0; i < sizeof(which) / sizeof(which[0]); i++) {
 		BPath basePath;
-		find_directory(which[i], &basePath);
-		basePath.Append("Screen Savers", true);
+		if (find_directory(which[i], &basePath) != B_OK)
+			continue;
+		else if (basePath.Append("Screen Savers", true) != B_OK)
+			continue;
 
 		BDirectory dir(basePath.Path());
 		BEntry entry;
@@ -712,8 +714,9 @@ ModulesView::PopulateScreenSaverList()
 			if (entry.GetName(name) != B_OK)
 				continue;
 
-			BPath path = basePath;
-			path.Append(name);
+			BPath path(basePath);
+			if (path.Append(name) != B_OK)
+				continue;
 
 			ScreenSaverItem* item = new ScreenSaverItem(name, path.Path());
 			fScreenSaversListView->AddItem(item);