Listing content had various pointer-related issues.
Signed-off-by: Manuel Bachmann <manuel.bachmann@iot.bzh>
STATIC json_object* list (AFB_request *request) { /* AFB_SESSION_CHECK */
STATIC json_object* list (AFB_request *request) { /* AFB_SESSION_CHECK */
char *result;
result = _rygel_list (request->context);
char *result;
result = _rygel_list (request->context);
- return jsonNewMessage(AFB_SUCCESS, result);
+ if (!result)
+ return jsonNewMessage(AFB_FAIL, "No content found in media server");
+
+ jresp = json_object_new_object();
+ json_object_object_add(jresp, "list", json_object_new_string (result));
+ return jresp;
}
STATIC json_object* ping (AFB_request *request) { /* AFB_SESSION_NONE */
}
STATIC json_object* ping (AFB_request *request) { /* AFB_SESSION_NONE */
void *media_server; /* handle to implementation (Rygel...) */
} mediaCtxHandleT;
void *media_server; /* handle to implementation (Rygel...) */
} mediaCtxHandleT;
+PUBLIC char* _rygel_list (mediaCtxHandleT *);
+
control_point = gupnp_control_point_new (context, URN_MEDIA_SERVER);
control_point = gupnp_control_point_new (context, URN_MEDIA_SERVER);
- g_signal_connect (control_point, "device-proxy-available",
- G_CALLBACK (_rygel_device_cb), ctx);
+ handler_cb = g_signal_connect (control_point, "device-proxy-available",
+ G_CALLBACK (_rygel_device_cb), ctx);
/* start searching for servers */
gssdp_resource_browser_set_active (GSSDP_RESOURCE_BROWSER (control_point), TRUE);
/* start searching for servers */
gssdp_resource_browser_set_active (GSSDP_RESOURCE_BROWSER (control_point), TRUE);
if (ctx->media_server)
break;
if (ctx->media_server)
break;
+ gettimeofday (&tv_now, NULL);
}
/* fail if we found no server */
if (!ctx->media_server)
}
/* fail if we found no server */
if (!ctx->media_server)
dev_ctx_c->context = NULL;
dev_ctx_c->device_info = NULL;
dev_ctx_c->content_dir = NULL;
dev_ctx_c->context = NULL;
dev_ctx_c->device_info = NULL;
dev_ctx_c->content_dir = NULL;
+ if (dev_ctx_c->content_res)
+ free (dev_ctx_c->content_res);
+ dev_ctx_c->content_res = NULL;
}
PUBLIC char* _rygel_list (mediaCtxHandleT *ctx) {
dev_ctx_T *dev_ctx_c = (dev_ctx_T*)ctx->media_server;
GUPnPServiceProxy *content_dir_proxy;
}
PUBLIC char* _rygel_list (mediaCtxHandleT *ctx) {
dev_ctx_T *dev_ctx_c = (dev_ctx_T*)ctx->media_server;
GUPnPServiceProxy *content_dir_proxy;
+ struct timeval tv_start, tv_now;
+ if (dev_ctx_c->content_res)
+ free (dev_ctx_c->content_res);
dev_ctx_c->content_res = NULL;
dev_ctx_c->content_res = NULL;
content_dir_proxy = GUPNP_SERVICE_PROXY (dev_ctx_c->content_dir);
gupnp_service_proxy_begin_action (content_dir_proxy, "Browse", _rygel_content_cb, dev_ctx_c,
content_dir_proxy = GUPNP_SERVICE_PROXY (dev_ctx_c->content_dir);
gupnp_service_proxy_begin_action (content_dir_proxy, "Browse", _rygel_content_cb, dev_ctx_c,
"SortCriteria", G_TYPE_STRING, "",
NULL);
"SortCriteria", G_TYPE_STRING, "",
NULL);
- while (!dev_ctx_c->content_res)
- g_main_context_iteration (dev_ctx_c->loop, FALSE);
+ gettimeofday (&tv_start, NULL);
+ gettimeofday (&tv_now, NULL);
+ while (tv_now.tv_sec - tv_start.tv_sec <= 5) {
+
+ g_main_context_iteration (dev_ctx_c->loop, FALSE);
+
+ if (dev_ctx_c->content_res)
+ break;
+ gettimeofday (&tv_now, NULL);
+ }
return dev_ctx_c->content_res;
}
return dev_ctx_c->content_res;
}
if (!content_dir)
return;
if (!content_dir)
return;
+ /* we have found Rygel ; stop looking for it... */
+ g_signal_handler_disconnect (point, handler_cb);
+
/* allocate the global array if it has not been not done */
if (!dev_ctx)
dev_ctx = (dev_ctx_T**) malloc (sizeof(dev_ctx_T));
/* allocate the global array if it has not been not done */
if (!dev_ctx)
dev_ctx = (dev_ctx_T**) malloc (sizeof(dev_ctx_T));
dev_ctx = (dev_ctx_T**) realloc (dev_ctx, (client_count+1)*sizeof(dev_ctx_T));
/* create an element for the client in the global array */
dev_ctx = (dev_ctx_T**) realloc (dev_ctx, (client_count+1)*sizeof(dev_ctx_T));
/* create an element for the client in the global array */
- dev_ctx[client_count] = malloc (sizeof(dev_ctx_T));
+ dev_ctx[client_count] = (dev_ctx_T*) malloc (sizeof(dev_ctx_T));
dev_ctx[client_count]->device_info = device_info;
dev_ctx[client_count]->content_dir = content_dir;
dev_ctx[client_count]->device_info = device_info;
dev_ctx[client_count]->content_dir = content_dir;
found += 4;
strncpy (subid, found, 32); subid[32] = '\0';
found += 4;
strncpy (subid, found, 32); subid[32] = '\0';
- gupnp_service_proxy_begin_action (content_dir_proxy, "Browse", _rygel_content_cb, NULL,
+ gupnp_service_proxy_begin_action (content_dir_proxy, "Browse", _rygel_content_cb, dev_ctx_c,
"ObjectID", G_TYPE_STRING, subid,
"BrowseFlag", G_TYPE_STRING, "BrowseDirectChildren",
"Filter", G_TYPE_STRING, "@childCount",
"ObjectID", G_TYPE_STRING, subid,
"BrowseFlag", G_TYPE_STRING, "BrowseDirectChildren",
"Filter", G_TYPE_STRING, "@childCount",
STATIC void _rygel_device_cb (GUPnPControlPoint *, GUPnPDeviceProxy *, gpointer);
STATIC void _rygel_content_cb (GUPnPServiceProxy *, GUPnPServiceProxyAction *, gpointer);
STATIC void _rygel_device_cb (GUPnPControlPoint *, GUPnPDeviceProxy *, gpointer);
STATIC void _rygel_content_cb (GUPnPServiceProxy *, GUPnPServiceProxyAction *, gpointer);
-static struct dev_ctx **dev_ctx = NULL;
static unsigned int client_count = 0;
static unsigned int client_count = 0;
+static struct dev_ctx **dev_ctx = NULL;
#endif /* MEDIA_RYGEL_H */
#endif /* MEDIA_RYGEL_H */