Bug #135
closedDocumentation should mention that lttng_event must be initialized to 0 to use the default values
100%
Description
If we do not initialize the struct lttng_event to 0, the lttng sessiond might return a fatal error when executing (if the value of the loglevel_type is set to an unknown value).
Documentation should mention that we can (have to ?) initialize to 0 the structure with :
struct lttng_event event = {0};
Or
struct lttng_event event; memset(&event, 0, sizeof(struct lttng_event)));
Updated by David Goulet almost 13 years ago
- Status changed from New to Confirmed
- Assignee set to Raphaël Beamonte
This is actually safer since this data is passed over the network and zero is "normally" uninitialized values.
However, the lttng ctl API enable-even should check mandatory field before sending it back to the session daemon and as of today, the loglevel check is not there.
Feel free to send a patch for developer documentation and loglevel check in liblttng-ctl enable-event call.
Updated by Yannick Brosseau almost 13 years ago
Should mention that all struct should be initialized to zéro before use
Updated by Anonymous almost 13 years ago
- Status changed from Confirmed to Resolved
- % Done changed from 0 to 100
Applied in changeset ece640c85fa5c2a96571aa54496f7b4414be91cd.