* Arnaldo Carvalho de Melo wrote:
> Em Tue, Jan 22, 2013 at 02:16:47PM +0100, Jiri Olsa escreveu:
> > On Mon, Jan 21, 2013 at 11:45:42PM -0500, Steven Rostedt wrote:
> > > This is only used in this function (the field_dynamic_elem_size() is
> > > only called here). Can we not add the
Em Tue, Jan 22, 2013 at 02:16:47PM +0100, Jiri Olsa escreveu:
> On Mon, Jan 21, 2013 at 11:45:42PM -0500, Steven Rostedt wrote:
> > This is only used in this function (the field_dynamic_elem_size() is
> > only called here). Can we not add the field->type_dyn, and just use a
> > local variable
Em Tue, Jan 22, 2013 at 02:16:47PM +0100, Jiri Olsa escreveu:
On Mon, Jan 21, 2013 at 11:45:42PM -0500, Steven Rostedt wrote:
This is only used in this function (the field_dynamic_elem_size() is
only called here). Can we not add the field-type_dyn, and just use a
local variable here. You
* Arnaldo Carvalho de Melo a...@redhat.com wrote:
Em Tue, Jan 22, 2013 at 02:16:47PM +0100, Jiri Olsa escreveu:
On Mon, Jan 21, 2013 at 11:45:42PM -0500, Steven Rostedt wrote:
This is only used in this function (the field_dynamic_elem_size() is
only called here). Can we not add the
On Mon, Jan 21, 2013 at 11:45:42PM -0500, Steven Rostedt wrote:
> On Mon, 2013-01-21 at 13:44 +0100, Jiri Olsa wrote:
SNIP
> > + for (i = 0; table[i].type; i++) {
> > + if (!strcmp(table[i].type, field->type_dyn))
> > + return table[i].size;
> > + }
> > +
> > +
On Mon, Jan 21, 2013 at 11:45:42PM -0500, Steven Rostedt wrote:
On Mon, 2013-01-21 at 13:44 +0100, Jiri Olsa wrote:
SNIP
+ for (i = 0; table[i].type; i++) {
+ if (!strcmp(table[i].type, field-type_dyn))
+ return table[i].size;
+ }
+
+ return 0;
+}
On Mon, 2013-01-21 at 13:44 +0100, Jiri Olsa wrote:
> Fixing the dynamic array format field parsing.
>
> Currently the event_read_fields function could segfault while parsing
> dynamic array other than string type. The reason is the event->pevent
> does not need to be set and gets dereferenced
Fixing the dynamic array format field parsing.
Currently the event_read_fields function could segfault while parsing
dynamic array other than string type. The reason is the event->pevent
does not need to be set and gets dereferenced unconditionaly.
Also adding proper initialization of
Fixing the dynamic array format field parsing.
Currently the event_read_fields function could segfault while parsing
dynamic array other than string type. The reason is the event-pevent
does not need to be set and gets dereferenced unconditionaly.
Also adding proper initialization of
On Mon, 2013-01-21 at 13:44 +0100, Jiri Olsa wrote:
Fixing the dynamic array format field parsing.
Currently the event_read_fields function could segfault while parsing
dynamic array other than string type. The reason is the event-pevent
does not need to be set and gets dereferenced
10 matches
Mail list logo