perl.h:175:16: error: 'my_perl' undeclared about dbd-mariadb

[rzl]

i try to add dTHX to free_embedded_options function , gmake success

int free_embedded_options(char ** options_list, int options_count)
{
  int i;
  dTHX;
  for (i= 0; i < options_count; i++)
  {
    if (options_list[i])
      free(options_list[i]);
  }
  free(options_list);

  return 1;
}

[pali]

This is incorrect. Memory which is going to be freed by free() call in free_embedded_options is allocated in fill_out_embedded_options by calloc() call.

Perl headers file replaces malloc() and free() calls by own memory management. But does not replaces calloc() call.

On some systems those functions just calls system's malloc() and free() functions, therefore everything is safe. (Seems that all Linux machines which I tested). But error which you got on Windows indicates that it really has own memory management (needs that dTHX;) and calling Perl's free() on memory allocated by Windows native calloc() would for sure cause memory corruptions...

Correct way is to stop using calloc() and free() and uses Perl's Newz() and Safefree() replacment.

[rzl]

though I don't know what you are talking about, replace free to Safefree , gmake success.
thank you .

int free_embedded_options(char ** options_list, int options_count)
{
  int i;

  for (i= 0; i < options_count; i++)
  {
    if (options_list[i])
      Safefree(options_list[i]);
  }
  Safefree(options_list);

  return 1;
}

[pali]

though I don't know what you are talking about

That for allocation of memory calloc must be avoided and instead Newz should be used. And for releasing memory allocated by Newz should be used Safefree call.

[pali]

Will you prepare a proper fix for it?

[pali]

Patch for guarding calloc for perl itself is in following RT ticket: https://rt.perl.org/Public/Bug/Display.html?id=133550

[pali]

Ok, I did in in pull request #94. Please check.

[rzl]

Ok, I did in in pull request #94. Please check.

git clone https://github.com/pali/DBD-MariaDB.git
perl Makefile.PL --mysql_config=c:/path/mysql_config.bat
gmake
gmake install

success

[pali]

Ok, therefore it is fixed, closing.