[tarantool-patches] Re: [PATCH v1 2/2] sql: fixed possible leak in sqlite3EndTable

From: Kirill Shcherbatov <kshcherbatov@xxxxxxxxxxxxx>
To: tarantool-patches@xxxxxxxxxxxxx, Vladislav Shpilevoy <v.shpilevoy@xxxxxxxxxxxxx>
Date: Fri, 13 Jul 2018 19:05:26 +0300

Thanks for the patch! See 1 comment below.

I see two returns below this line. Why have not you replaced
them with goto cleanup?

It is not important after space_def_rebuild. But I re factored code with goto
cleanup
to be consistent.

References:
- [tarantool-patches] [PATCH v1 0/2] sql: restrict nullable action definitions
  - From: Kirill Shcherbatov
- [tarantool-patches] [PATCH v1 2/2] sql: fixed possible leak in sqlite3EndTable
  - From: Kirill Shcherbatov
- [tarantool-patches] Re: [PATCH v1 2/2] sql: fixed possible leak in sqlite3EndTable
  - From: Vladislav Shpilevoy

Other related posts:

» [tarantool-patches] [PATCH v1 2/2] sql: fixed possible leak in sqlite3EndTable- Kirill Shcherbatov
» [tarantool-patches] Re: [PATCH v1 2/2] sql: fixed possible leak in sqlite3EndTable- Vladislav Shpilevoy
» [tarantool-patches] Re: [PATCH v1 2/2] sql: fixed possible leak in sqlite3EndTable - Kirill Shcherbatov