PengZheng opened a new issue, #686:
URL: https://github.com/apache/celix/issues/686
Currently, the overall coverage of the framework library is around 85%.
Utilizing error injector, we shall try to increase the code coverage to 90%
or more.
This should happen after deprecated APIs have been removed.
I believe by increasing the overall coverage, we can eliminate many corner
case bugs like the following:
```C
/**
* @brief Register a service to the Celix framework.
*
* Note: Please use the celix_bundleContext_registerServiceAsync instead.
*
* @param ctx The bundle context
* @param svc the service object. Normally a pointer to a service struct
(i.e. a struct with function pointers)
* @param serviceName the service name, cannot be NULL
* @param properties The meta properties associated with the service. The
service registration will take ownership of the properties (i.e. no destroy
needed)
* @return The serviceId (>=0) or -1 if the registration was unsuccessful.
*/
CELIX_FRAMEWORK_EXPORT long
celix_bundleContext_registerService(celix_bundle_context_t *ctx, void *svc,
const char* serviceName, celix_properties_t *properties);
```
`celix_bundleContext_registerService` will leak `properties` when
`serviceName == NULL`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@celix.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
