projects
/
babeltrace.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix: --clock-force-correlate to handle trace collections gathered from various nodes
[babeltrace.git]
/
lib
/
context.c
diff --git
a/lib/context.c
b/lib/context.c
index e2c1739c41ff548a21ae13a3e62f983779590e5b..0f727a180a53c5834c07ff2830544b61b7c73f97 100644
(file)
--- a/
lib/context.c
+++ b/
lib/context.c
@@
-30,8
+30,8
@@
#include <stdlib.h>
#include <string.h>
#include <assert.h>
#include <stdlib.h>
#include <string.h>
#include <assert.h>
+#include <errno.h>
-#include <fts.h>
#include <fcntl.h> /* For O_RDONLY */
#include <glib.h>
#include <fcntl.h> /* For O_RDONLY */
#include <glib.h>
@@
-69,6
+69,9
@@
int bt_context_add_trace(struct bt_context *ctx, const char *path,
struct bt_trace_handle *handle;
int ret;
struct bt_trace_handle *handle;
int ret;
+ if (!ctx || !format_name || (!path && !stream_list))
+ return -EINVAL;
+
fmt = bt_lookup_format(g_quark_from_string(format_name));
if (!fmt) {
fprintf(stderr, "[error] [Context] Format \"%s\" unknown.\n\n",
fmt = bt_lookup_format(g_quark_from_string(format_name));
if (!fmt) {
fprintf(stderr, "[error] [Context] Format \"%s\" unknown.\n\n",
@@
-136,13
+139,17
@@
end:
return ret;
}
return ret;
}
-
void
bt_context_remove_trace(struct bt_context *ctx, int handle_id)
+
int
bt_context_remove_trace(struct bt_context *ctx, int handle_id)
{
struct bt_trace_handle *handle;
{
struct bt_trace_handle *handle;
+ if (!ctx)
+ return -EINVAL;
+
handle = g_hash_table_lookup(ctx->trace_handles,
(gpointer) (unsigned long) handle_id);
handle = g_hash_table_lookup(ctx->trace_handles,
(gpointer) (unsigned long) handle_id);
- assert(handle != NULL);
+ if (!handle)
+ return -ENOENT;
/* Remove from containers */
trace_collection_remove(ctx->tc, handle->td);
/* Remove from containers */
trace_collection_remove(ctx->tc, handle->td);
@@
-152,12
+159,13
@@
void bt_context_remove_trace(struct bt_context *ctx, int handle_id)
/* Remove and free the handle */
g_hash_table_remove(ctx->trace_handles,
(gpointer) (unsigned long) handle_id);
/* Remove and free the handle */
g_hash_table_remove(ctx->trace_handles,
(gpointer) (unsigned long) handle_id);
-
+ return 0;
}
static
void bt_context_destroy(struct bt_context *ctx)
{
}
static
void bt_context_destroy(struct bt_context *ctx)
{
+ assert(ctx);
finalize_trace_collection(ctx->tc);
/* Remote all traces. The g_hash_table_destroy will call
finalize_trace_collection(ctx->tc);
/* Remote all traces. The g_hash_table_destroy will call
@@
-173,11
+181,13
@@
void bt_context_destroy(struct bt_context *ctx)
void bt_context_get(struct bt_context *ctx)
{
void bt_context_get(struct bt_context *ctx)
{
+ assert(ctx);
ctx->refcount++;
}
void bt_context_put(struct bt_context *ctx)
{
ctx->refcount++;
}
void bt_context_put(struct bt_context *ctx)
{
+ assert(ctx);
ctx->refcount--;
if (ctx->refcount == 0)
bt_context_destroy(ctx);
ctx->refcount--;
if (ctx->refcount == 0)
bt_context_destroy(ctx);
This page took
0.023335 seconds
and
4
git commands to generate.