- bug : https://bugs.launchpad.net/linaro-powerdebug/+bug/1014667
Signed-off-by: Sanjay Singh Rawat <sanjay.ra...@linaro.org>
---
display.c | 4 +++-
powerdebug.c | 2 +-
regulator.c | 4 +++-
tree.c | 4 +++-
4 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/display.c b/display.c
index f06387c..39a8e71 100644
--- a/display.c
+++ b/display.c
@@ -593,8 +593,10 @@ int display_register(int win, struct display_ops *ops)
{
size_t array_size = sizeof(windata) / sizeof(windata[0]);
- if (win < 0 || win >= array_size)
+ if (win < 0 || win >= array_size) {
+ printf("error: invalid window");
return -1;
+ }
windata[win].ops = ops;
diff --git a/powerdebug.c b/powerdebug.c
index 073fd42..ca04fab 100644
--- a/powerdebug.c
+++ b/powerdebug.c
@@ -243,7 +243,7 @@ int main(int argc, char **argv)
}
if (regulator_init()) {
- printf("not enough memory to allocate regulators info\n");
+ printf("failed to initialize regulator\n");
options->regulators = false;
}
diff --git a/regulator.c b/regulator.c
index 0da924a..68c686f 100644
--- a/regulator.c
+++ b/regulator.c
@@ -218,8 +218,10 @@ static int fill_regulator_cb(struct tree *t, void *data)
struct regulator_info *reg;
reg = regulator_alloc();
- if (!reg)
+ if (!reg) {
+ printf("error: unable to allocate memory for regulator\n");
return -1;
+ }
t->private = reg;
/* we skip the root node but we set it expanded for its children */
diff --git a/tree.c b/tree.c
index d331c60..686e846 100644
--- a/tree.c
+++ b/tree.c
@@ -121,8 +121,10 @@ static int tree_scan(struct tree *tree, tree_filter_t
filter, bool follow)
int ret = 0;
dir = opendir(tree->path);
- if (!dir)
+ if (!dir) {
+ printf("error: unable to open directory %s\n", tree->path);
return -1;
+ }
while (!readdir_r(dir, &dirent, &direntp)) {
--
1.7.9.5
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev
