[tarantool-patches] Re: [PATCH] msgpackffi.decode can now be assigned to buf.rpos

  • From: Kirill Yukhin <kyukhin@xxxxxxxxxxxxx>
  • To: Alexander Turenko <alexander.turenko@xxxxxxxxxxxxx>
  • Date: Tue, 31 Dec 2019 10:51:30 +0300

Hello,

On 31 дек 06:56, Alexander Turenko wrote:

On Mon, Dec 30, 2019 at 10:53:26AM +0300, Kirill Yukhin wrote:
Hello,

On 12 сен 20:44, Maria wrote:
Function decode of module msgpackffi was passing
value of type const unsigned char * to a C function
that accepts arguments of type const char *.

Closes #3926

Test added
---
 src/lua/msgpackffi.lua           | 8 +++++---
 test/app-tap/msgpackffi.test.lua | 6 ++++++
 2 files changed, 11 insertions(+), 3 deletions(-)

I've checked your patch into master.

I would reconsider it as bug and push everywhere (both patches).

It seems that significant part of the problem is just because someone
forgot to cast (unsigned char *) back to (char *) before returning.

What do you think, Kirill?
Yup.

--
Regards, Kirill Yukhin

Other related posts: