[Tarantool-patches] [PATCH] cppcheck: box/alter.cc null pointer dereference

Sergey Kaplun skaplun at tarantool.org
Mon Sep 21 23:37:06 MSK 2020


Hi! Thanks for the review!

On 21.09.20, Vladislav Shpilevoy wrote:
> Hi! Thanks for the patch!
> 
> Commit title should be a subsystem name + description. Since we
> don't have 'cppcheck' subsystem, I propose to use 'alter'.
> 
> LGTM except the commit title.

I've updated commit message to:

```
alter: box/alter.cc null pointer dereference

Found and fixed Null pointer dereference with cppcheck:

[src/box/alter.cc:395]: (error) Null pointer dereference
```
and pushed it to the branch.

-- 
Best regards,
Sergey Kaplun


More information about the Tarantool-patches mailing list