cwPianoScore.cpp : Copy object values into internal data structure with strncpy() instead of memcpy() to eliminate possible memory corruption.

Fixed memory leaks.
This commit is contained in:
kevin 2021-12-29 21:46:54 -05:00
parent f321af21a2
commit 2601c629e2

View File

@ -88,13 +88,13 @@ namespace cw
if( sci_pitch != nullptr )
memcpy(e->sci_pitch,sci_pitch,sizeof(e->sci_pitch));
strncpy(e->sci_pitch,sci_pitch,sizeof(e->sci_pitch));
if( dmark != nullptr )
memcpy(e->dmark,dmark,sizeof(e->dmark));
strncpy(e->dmark,dmark,sizeof(e->dmark));
if( grace_mark != nullptr )
memcpy(e->grace_mark,grace_mark,sizeof(e->grace_mark));
strncpy(e->grace_mark,grace_mark,sizeof(e->grace_mark));
// assign the UID
e->uid = i;
@ -150,6 +150,10 @@ cw::rc_t cw::score::create( handle_t& hRef, const char* fn )
rc = create(hRef,cfg);
errLabel:
if( cfg != nullptr )
cfg->free();
return rc;
}