X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=src%2Fbackend.c;h=613de246a0298ae76e4e829aa01d8a3b9aab8e8d;hb=6d930b4159c2df2a500610f50c6df360f4dcd29a;hp=c2a83cfedfe4f48a831696ee1b58dfe0b99a10a4;hpb=9e60a31fb954493754770fe2192db74b947c6867;p=libsigrok.git diff --git a/src/backend.c b/src/backend.c index c2a83cfe..613de246 100644 --- a/src/backend.c +++ b/src/backend.c @@ -79,10 +79,10 @@ * * Initializing and shutting down libsigrok. * - * Before using any of the libsigrok functionality (except - * sr_log_loglevel_set() and sr_log_opts_set()), sr_init() must - * be called to initialize the library, which will return a struct sr_context - * when the initialization was successful. + * Before using any of the libsigrok functionality (except for + * sr_log_loglevel_set()), sr_init() must be called to initialize the + * library, which will return a struct sr_context when the initialization + * was successful. * * When libsigrok functionality is no longer needed, sr_exit() should be * called, which will (among other things) free the struct sr_context. @@ -122,18 +122,24 @@ /** * Sanity-check all libsigrok drivers. * + * @param[in] ctx Pointer to a libsigrok context struct. Must not be NULL. + * * @retval SR_OK All drivers are OK * @retval SR_ERR One or more drivers have issues. + * @retval SR_ERR_ARG Invalid argument. */ -static int sanity_check_all_drivers(void) +static int sanity_check_all_drivers(const struct sr_context *ctx) { int i, errors, ret = SR_OK; struct sr_dev_driver **drivers; const char *d; + if (!ctx) + return SR_ERR_ARG; + sr_spew("Sanity-checking all drivers."); - drivers = sr_driver_list(); + drivers = sr_driver_list(ctx); for (i = 0; drivers[i]; i++) { errors = 0; @@ -375,13 +381,25 @@ SR_API int sr_init(struct sr_context **ctx) { int ret = SR_ERR; struct sr_context *context; + struct sr_dev_driver ***lists, **drivers; + GArray *array; if (!ctx) { sr_err("%s(): libsigrok context was NULL.", __func__); return SR_ERR; } - if (sanity_check_all_drivers() < 0) { + context = g_malloc0(sizeof(struct sr_context)); + + /* Generate ctx->driver_list at runtime. */ + array = g_array_new(TRUE, FALSE, sizeof(struct sr_dev_driver *)); + for (lists = drivers_lists; *lists; lists++) + for (drivers = *lists; *drivers; drivers++) + g_array_append_val(array, *drivers); + context->driver_list = (struct sr_dev_driver **)array->data; + g_array_free(array, FALSE); + + if (sanity_check_all_drivers(context) < 0) { sr_err("Internal driver error(s), aborting."); return ret; } @@ -401,9 +419,6 @@ SR_API int sr_init(struct sr_context **ctx) return ret; } - /* + 1 to handle when struct sr_context has no members. */ - context = g_malloc0(sizeof(struct sr_context) + 1); - #ifdef HAVE_LIBUSB_1_0 ret = libusb_init(&context->libusb_ctx); if (LIBUSB_SUCCESS != ret) { @@ -417,9 +432,10 @@ SR_API int sr_init(struct sr_context **ctx) context = NULL; ret = SR_OK; +#ifdef HAVE_LIBUSB_1_0 done: - if (context) - g_free(context); +#endif + g_free(context); return ret; } @@ -440,13 +456,13 @@ SR_API int sr_exit(struct sr_context *ctx) return SR_ERR; } - sr_hw_cleanup_all(); + sr_hw_cleanup_all(ctx); #ifdef HAVE_LIBUSB_1_0 libusb_exit(ctx->libusb_ctx); #endif - g_free(sr_driver_list()); + g_free(sr_driver_list(ctx)); g_free(ctx); return SR_OK;