We should not free() a dlopened pointer.
Internal plugin handle was renamed, adapt radio plugin.
Signed-off-by: Manuel Bachmann <manuel.bachmann@iot.bzh>
STATIC json_object* power (AFB_request *request) { /* AFB_SESSION_CHECK */
STATIC json_object* power (AFB_request *request) { /* AFB_SESSION_CHECK */
- pluginHandleT *handle = (pluginHandleT*)request->plugin;
+ pluginHandleT *handle = (pluginHandleT*)request->handle;
radioCtxHandleT *ctx = (radioCtxHandleT*)request->context;
const char *value = getQueryValue (request, "value");
json_object *jresp;
radioCtxHandleT *ctx = (radioCtxHandleT*)request->context;
const char *value = getQueryValue (request, "value");
json_object *jresp;
STATIC void scanDirectory(char *dirpath, int dirfd, AFB_plugin **plugins, int *count) {
DIR *dir;
void *libso;
STATIC void scanDirectory(char *dirpath, int dirfd, AFB_plugin **plugins, int *count) {
DIR *dir;
void *libso;
- struct dirent *pluginDir;
+ struct dirent pluginDir, *result;
AFB_plugin* (*pluginRegisterFct)(void);
char pluginPath[255];
AFB_plugin* (*pluginRegisterFct)(void);
char pluginPath[255];
// Open Directory to scan over it
dir = fdopendir (dirfd);
if (dir == NULL) {
// Open Directory to scan over it
dir = fdopendir (dirfd);
if (dir == NULL) {
}
if (verbose) fprintf (stderr, "Scanning dir=[%s] for plugins\n", dirpath);
}
if (verbose) fprintf (stderr, "Scanning dir=[%s] for plugins\n", dirpath);
- while ((pluginDir = readdir(dir)) != NULL) {
-
+ for (;;) {
+ readdir_r(dir, &pluginDir, &result);
+ if (result == NULL) break;
+
// Loop on any contained directory
// Loop on any contained directory
- if ((pluginDir->d_type == DT_DIR) && (pluginDir->d_name[0] != '.')) {
- int fd = openat (dirfd, pluginDir->d_name, O_DIRECTORY);
+ if ((pluginDir.d_type == DT_DIR) && (pluginDir.d_name[0] != '.')) {
+ int fd = openat (dirfd, pluginDir.d_name, O_DIRECTORY);
char newpath[255];
strncpy (newpath, dirpath, sizeof(newpath));
strncat (newpath, "/", sizeof(newpath));
char newpath[255];
strncpy (newpath, dirpath, sizeof(newpath));
strncat (newpath, "/", sizeof(newpath));
- strncat (newpath, pluginDir->d_name, sizeof(newpath));
+ strncat (newpath, pluginDir.d_name, sizeof(newpath));
scanDirectory (newpath, fd, plugins, count);
close (fd);
scanDirectory (newpath, fd, plugins, count);
close (fd);
// This is a file but not a plugin let's move to next directory element
// This is a file but not a plugin let's move to next directory element
- if (!strstr (pluginDir->d_name, ".so")) continue;
+ if (!strstr (pluginDir.d_name, ".so")) continue;
// This is a loadable library let's check if it's a plugin
// This is a loadable library let's check if it's a plugin
- snprintf (pluginPath, sizeof(pluginPath), "%s/%s", dirpath, pluginDir->d_name);
+ snprintf (pluginPath, sizeof(pluginPath), "%s/%s", dirpath, pluginDir.d_name);
libso = dlopen (pluginPath, RTLD_NOW | RTLD_LOCAL);
// Load fail we ignore this .so file
if (!libso) {
libso = dlopen (pluginPath, RTLD_NOW | RTLD_LOCAL);
// Load fail we ignore this .so file
if (!libso) {
- fprintf(stderr, "[%s] is not loadable, continuing...\n", pluginDir->d_name);
+ fprintf(stderr, "[%s] is not loadable, continuing...\n", pluginDir.d_name);
pluginRegisterFct = dlsym (libso, "pluginRegister");
pluginRegisterFct = dlsym (libso, "pluginRegister");
if (!pluginRegisterFct) {
if (!pluginRegisterFct) {
- fprintf(stderr, "[%s] is not an AFB plugin, continuing...\n", pluginDir->d_name);
+ fprintf(stderr, "[%s] is not an AFB plugin, continuing...\n", pluginDir.d_name);
- if (verbose) fprintf(stderr, "[%s] is a valid AFB plugin, loading pos[%d]\n", pluginDir->d_name, *count);
+ if (verbose) fprintf(stderr, "[%s] is a valid AFB plugin, loading pos[%d]\n", pluginDir.d_name, *count);
plugins[*count] = (AFB_plugin *) malloc (sizeof(AFB_plugin));
plugins[*count] = (**pluginRegisterFct)();
*count = *count +1;
plugins[*count] = (AFB_plugin *) malloc (sizeof(AFB_plugin));
plugins[*count] = (**pluginRegisterFct)();
*count = *count +1;