On Thu, Apr 5, 2018, at 14:09, Stefan Schmidt wrote: > Hello Shinwoo. > > > On 04/05/2018 12:42 PM, Shinwoo Kim wrote: > > kimcinoo pushed a commit to branch master. > > > > http://git.enlightenment.org/core/efl.git/commit/?id=3cd2243028f80960d3fe0b464f84514f609bab51 > > > > commit 3cd2243028f80960d3fe0b464f84514f609bab51 > > Author: Shinwoo Kim <kimci...@gmail.com> > > Date: Thu Apr 5 13:18:03 2018 +0900 > > > > Fix static analysis result > > > > [Dereference after null check] > > > > (1) src/lib/ecore/ecore_main.c > > - _efl_loop_handler_efl_object_finalize checks if pd->loop_data is > > NULL. > > After that, _handler_reset > _handler_clear > > > _ecore_main_fd_handler_del > > > _ecore_main_fdh_pool_del is directly dereferencing pd->pool_data. > > - _efl_loop_handler_efl_object_parent_set checks if pd->loop_data as > > well. > > Then it calls _handler_reset as well. > > > > (2) src/lib/ecore_wayland/ecore_wl_dnd.c > > - ecore_wl_dnd_selection_set checks if t - result of wl_array_add - > > is NULL. > > And it is dereferecing t directly for wl_data_source_offer. > > > > (3) src/lib/elementary/efl_ui_dnd.c > > - Third parameter const char *data could be NULL. > > In this case strlen dereferences NULL. The data should be non NULL > > value. > > I have checked this with Mr. Thiep Ha. > > > > (4) src/lib/evas/canvas/evas_object_inform.c > > - _efl_canvas_object_efl_gfx_stack_stack_below checks if obj->layer is > > NULL. > > So it could call evas_object_inform_call_call_restack which is > > dereferencing > > obj->layer directly. > > --- > > src/lib/ecore/ecore_main.c | 6 ++++++ > > src/lib/ecore_wayland/ecore_wl_dnd.c | 7 +++++-- > > src/lib/elementary/efl_ui_dnd.c | 1 + > > src/lib/evas/canvas/evas_object_inform.c | 3 ++- > > 4 files changed, 14 insertions(+), 3 deletions(-) > > > > These are 4 different problems you are fixing in 4 different library parts. > > It would be appreciated if you could have separate patches for each > problem you are fixing.
Also, please follow the commit message conventions. D5 > > regards > Stefan Schmidt > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel