diff options
| author |   Romain Forlot <romain.forlot@iot.bzh> | 2018-05-16 01:23:21 +0200 |
|---|---|---|
| committer | Romain Forlot <romain.forlot@iot.bzh> | 2018-05-17 17:58:26 +0200 |
| commit | 3899adeec8f464c262d9cf64d4b7e10eebdb6223 (patch) | |
| tree | b0eac5075c6e8546d5217762dfbcfd2aa488f825 | |
| parent | 2b6c0eb7d0df2bf818c439ee51b144a5e8d3216a (diff) | |
Improve reliability and function calls
Don't calls two times the same function.
As well as avoiding using the function return to
get its length
Change-Id: I26c37f9a376b2c243ffd7a5ad0ddf935c8b5d0e9
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
| -rw-r--r-- | ctl-lib/ctl-config.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ctl-lib/ctl-config.c b/ctl-lib/ctl-config.c index 5503db1..cac8c75 100644 --- a/ctl-lib/ctl-config.c +++ b/ctl-lib/ctl-config.c @@ -44,8 +44,10 @@ int CtlConfigMagicNew() { json_object* CtlConfigScan(const char *dirList, const char *prefix) { char controlFile [CONTROL_MAXPATH_LEN]; + const char *binderName = GetBinderName(); + strncpy(controlFile, prefix, strlen(prefix)+1); - strncat(controlFile, GetBinderName(), strlen(GetBinderName())); + strncat(controlFile, binderName, strlen(binderName)); // search for default dispatch config file json_object* responseJ = ScanForConfig(dirList, CTL_SCAN_RECURSIVE, controlFile, ".json"); @@ -112,7 +114,7 @@ int CtlConfigExec(AFB_ApiT apiHandle, CtlConfigT *ctlConfig) { // load static LUA utilities LuaConfigExec(apiHandle); #endif - + // Loop on every section and process config int errcount=0; for (int idx = 0; ctlConfig->sections[idx].key != NULL; idx++) { |