On Tue, Mar 29, 2016 at 7:19 PM, <vinodk...@apache.org> wrote: > --- a/src/sched/sched.cpp > +++ b/src/sched/sched.cpp > @@ -1808,6 +1808,10 @@ MesosSchedulerDriver::~MesosSchedulerDriver() > delete process; > } > > + if (credential != NULL) { > + delete credential; > + }
`delete` of a NULL pointer is safe, so I would vote for removing the `if`. Neil