1283032 - [Static Analysis][Uninitialized pointer field] In constructor RunnableMethod

Status: RESOLVED FIXED

(Core :: IPC, defect, P1)

Description

[Andi [:andi]]

The Static Analysis tool Coverity detected that member variables |obj_|, |meth_| and |params_| are not initialised in the default constructor. They are attributed values in the Init function that follows the constructor but though in order to integrate clang-plugin with uninitialised member checker we should move the initialisation in the constructor.

Comment 1

[Andi [:andi]]

Attachment: Bug 1283032 - initialize members from RunnableMethod in constructor.

Review commit: https://reviewboard.mozilla.org/r/61216/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/61216/

Comment 2

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Comment on attachment 8766243 [details]
Bug 1283032 - initialize members from RunnableMethod in constructor.

https://reviewboard.mozilla.org/r/61216/#review58460

::: ipc/glue/TaskFactory.h:66
(Diff revision 1)
>    template <class Method>
>    inline already_AddRefed<Runnable> NewRunnableMethod(Method method) {
>      typedef TaskWrapper<RunnableMethod<Method, Tuple0> > TaskWrapper;
>  
> -    RefPtr<TaskWrapper> task = new TaskWrapper(this);
> -    task->Init(object_, method, base::MakeTuple());
> +    RefPtr<TaskWrapper> task = new TaskWrapper(this, object_, method,
> +      base::MakeTuple());

This should be indented to line up witht he opening paren on the previous line.

::: ipc/glue/TaskFactory.h:76
(Diff revision 1)
>    template <class Method, class A>
>    inline already_AddRefed<Runnable> NewRunnableMethod(Method method, const A& a) {
>      typedef TaskWrapper<RunnableMethod<Method, Tuple1<A> > > TaskWrapper;
>  
> -    RefPtr<TaskWrapper> task = new TaskWrapper(this);
> -    task->Init(object_, method, base::MakeTuple(a));
> +    RefPtr<TaskWrapper> task = new TaskWrapper(this, object_, method,
> +      base::MakeTuple(a));

Same here.

::: ipc/glue/TaskFactory.h:88
(Diff revision 1)
>    class RunnableMethod : public Runnable {
>     public:
> -    RunnableMethod() { }
> +    RunnableMethod(T* obj, Method meth, const Params& params)
> +      : obj_(obj)
> +      , meth_(meth)
> +      , params_(params){

There should be a space between the closing paren and the opening brace.

Comment 3

[Andi [:andi]]

Comment on attachment 8766243 [details]
Bug 1283032 - initialize members from RunnableMethod in constructor.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/61216/diff/1-2/

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/607b4d01aaee
initialize members from RunnableMethod in constructor. r=billm

Comment 5

[Gregory Szorc [:gps]]

https://hg.mozilla.org/mozilla-central/rev/607b4d01aaee