[PATCH 2/2] boot: iterate over directories in alphabetical order
Sascha Hauer
s.hauer at pengutronix.de
Mon May 12 23:23:40 PDT 2014
When iterating over directories in order to find boot scripts do
this alphabetically to get a predictable order. This can be done
with glob() rather than readdir().
Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
---
commands/boot.c | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/commands/boot.c b/commands/boot.c
index 468d473..a23ffb1 100644
--- a/commands/boot.c
+++ b/commands/boot.c
@@ -23,6 +23,7 @@
#include <malloc.h>
#include <clock.h>
#include <boot.h>
+#include <glob.h>
#include <menu.h>
#include <fs.h>
#include <complete.h>
@@ -129,10 +130,10 @@ static int bootscript_create_entry(struct blspec *blspec, const char *name)
static int bootscript_scan_path(struct blspec *blspec, const char *path)
{
struct stat s;
- DIR *dir;
- struct dirent *d;
- int ret;
+ char *files;
+ int ret, i;
int found = 0;
+ glob_t globb;
ret = stat(path, &s);
if (ret)
@@ -145,25 +146,24 @@ static int bootscript_scan_path(struct blspec *blspec, const char *path)
return 1;
}
- dir = opendir(path);
- if (!dir)
- return -errno;
+ files = asprintf("%s/*", path);
+
+ glob(files, 0, NULL, &globb);
- while ((d = readdir(dir))) {
- char *bootscript_path;
+ for (i = 0; i < globb.gl_pathc; i++) {
+ char *bootscript_path = globb.gl_pathv[i];;
- if (*d->d_name == '.')
+ if (*basename(bootscript_path) == '.')
continue;
- bootscript_path = asprintf("%s/%s", path, d->d_name);
bootscript_create_entry(blspec, bootscript_path);
found++;
- free(bootscript_path);
}
- ret = found;
+ globfree(&globb);
+ free(files);
- closedir(dir);
+ ret = found;
return ret;
}
--
2.0.0.rc0
More information about the barebox
mailing list