Extension API: in parameter pair "char* buf" "int buf_sz", size is incorrect

[juliansader]

If an extension registers a function with a pair of parameters named "const char* something" and "int something_sz", REAPER recognizes these parameters as a linked pair. The script only has to supply the char* argument, and REAPER itself supplies the value of something_sz.

However, the value that REAPER supplies doesn't seem to be correct or useful. It seems to be calculated as one less than the number of characters before the first \0 char (unless \0 is the first character, in which case it is calculated differently).

It would be much more useful if REAPER passes the exact char length of the string (including all internal and terminal \0 chars). This would allow the extension to read the entire string without fears of memory access violations.

At present, in order for the extension to know the true length of a string argument, the script/user has to supply the length as a separate argument.

[schwa]

A string in many languages including C/C++ (but not Lua) is just a pointer to a string of characters, with no intrinsic length information. The standard C strlen function simply counts characters until the first 0 is seen. If a string contains more information beyond the first 0 character, that has to be specified by whoever created the string, there's no way for the receiver to know.

[nofish]

This helps maybe?
https://forum.cockos.com/showthread....08#post2055208

edit:
Sorry, that's about out parameters, not in parameters.

[Justin]

Edit: ah yes, there's some bugs in this, fixing!

[cfillion]

Scripts are allowed to set the size parameter bigger than the buffer:

// void Foo(const char *buf, int buf_sz)

reaper.Foo('a\0b\0', 1<<31) -- buffer over-read/crash