static void viewer_stream_destroy(struct relay_viewer_stream *vstream)
{
+ lttng_trace_chunk_put(vstream->stream_file.trace_chunk);
free(vstream->path_name);
free(vstream->channel_name);
free(vstream);
enum lttng_viewer_seek seek_t)
{
struct relay_viewer_stream *vstream = NULL;
- const bool acquired_reference = lttng_trace_chunk_get(trace_chunk);
ASSERT_LOCKED(stream->lock);
- if (!acquired_reference) {
- goto error;
- }
vstream = zmalloc(sizeof(*vstream));
if (!vstream) {
goto error;
}
+ if (trace_chunk) {
+ const bool acquired_reference = lttng_trace_chunk_get(
+ trace_chunk);
+
+ assert(acquired_reference);
+ }
+
vstream->stream_file.trace_chunk = trace_chunk;
- trace_chunk = NULL;
vstream->path_name = lttng_strndup(stream->path_name, LTTNG_VIEWER_PATH_MAX);
if (vstream->path_name == NULL) {
PERROR("relay viewer path_name alloc");
if (stream->is_metadata && stream->trace->viewer_metadata_stream) {
ERR("Cannot attach viewer metadata stream to trace (busy).");
- goto error_unlock;
+ goto error;
}
switch (seek_t) {
tracefile_array_get_seq_head(stream->tfa) + 1;
break;
default:
- goto error_unlock;
+ goto error;
}
/*
*/
if (stream->index_file == NULL) {
vstream->index_file = NULL;
- } else {
+ } else if (vstream->stream_file.trace_chunk) {
const uint32_t connection_major = stream->trace->session->major;
const uint32_t connection_minor = stream->trace->session->minor;
enum lttng_trace_chunk_status chunk_status;
if (chunk_status == LTTNG_TRACE_CHUNK_STATUS_NO_FILE) {
vstream->index_file = NULL;
} else {
- goto error_unlock;
+ goto error;
}
}
}
* If we never received a data file for the current stream, delay the
* opening, otherwise open it right now.
*/
- if (stream->file) {
+ if (stream->file && vstream->stream_file.trace_chunk) {
int ret;
char file_path[LTTNG_PATH_MAX];
enum lttng_trace_chunk_status status;
vstream->current_tracefile_id, NULL, file_path,
sizeof(file_path));
if (ret < 0) {
- goto error_unlock;
+ goto error;
}
status = lttng_trace_chunk_open_fs_handle(
O_RDONLY, 0, &vstream->stream_file.handle,
true);
if (status != LTTNG_TRACE_CHUNK_STATUS_OK) {
- goto error_unlock;
+ goto error;
}
}
lseek_ret = fs_handle_seek(
vstream->index_file->file, 0, SEEK_END);
if (lseek_ret < 0) {
- goto error_unlock;
+ goto error;
}
}
if (stream->is_metadata) {
vstream);
}
+ vstream->last_seen_rotation_count = stream->completed_rotation_count;
+
/* Globally visible after the add unique. */
lttng_ht_node_init_u64(&vstream->stream_n, stream->stream_handle);
urcu_ref_init(&vstream->ref);
return vstream;
-error_unlock:
- pthread_mutex_unlock(&stream->lock);
error:
if (vstream) {
viewer_stream_destroy(vstream);
}
- if (trace_chunk && acquired_reference) {
- lttng_trace_chunk_put(trace_chunk);
- }
return NULL;
}