Skip to content

Set of various fixes #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
pierrewillenbrockdfki wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Set of various fixes #37

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
32bd250
fix: Fix some memory leaks
pierrewillenbrockdfki Jan 9, 2023
106ee0f
fix: Special case bool and (u)int8_t when writing config strings
pierrewillenbrockdfki Jan 15, 2026
9bd9ce8
fix: No need to cast (u)int8_t to (u)int when finally casting to double
pierrewillenbrockdfki Jan 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 17 additions & 7 deletions src/ConfigurationHelper.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -318,9 +318,9 @@ bool ConfigurationHelper::applyConfOnTyplibValue(Typelib::Value &value, const Co

for(const std::shared_ptr<ConfigValue> val: array->getValues())
{

//TODO check, this may be a memory leak
Typelib::Value v(new uint8_t[indirect.getSize()], indirect);
std::vector<uint8_t> storage;
storage.resize(indirect.getSize());
Typelib::Value v(storage.data(), indirect);
Typelib::init(v);
Typelib::zero(v);

Expand Down Expand Up @@ -416,7 +416,10 @@ bool ConfigurationHelper::applyConfigValueOnDSB(RTT::base::DataSourceBase::share

//write value back
typelibTransport->writeDataSource(*dsb, handle);


//delete all referenced memory to avoid memory leak
Typelib::destroy(dest);

//destroy handle to avoid memory leak
typelibTransport->deleteHandle(handle);

Expand Down Expand Up @@ -554,7 +557,12 @@ bool ConfigurationHelper::registerOverride(const std::string& taskName, Configur
}

YAML::Emitter &toYAML(YAML::Emitter &out, const Typelib::Numeric &type, const Typelib::Value &value){

if (type.getName() == "/bool")
{
//the /bool type must be output as "true" or "false"
out << (*static_cast<bool *>(value.getData())?"true":"false");
return out;
}
switch(type.getNumericCategory())
{
case Typelib::Numeric::Float:
Expand All @@ -571,7 +579,8 @@ YAML::Emitter &toYAML(YAML::Emitter &out, const Typelib::Numeric &type, const Ty
switch(type.getSize())
{
case sizeof(int8_t):
out << *(static_cast<int8_t *>(value.getData()));
//need to cast uint8_t to int so it is not interpreted as character but as number
out << static_cast<int>(*(static_cast<int8_t *>(value.getData())));
break;
case sizeof(int16_t):
out << *(static_cast<int16_t *>(value.getData()));
Expand All @@ -593,7 +602,8 @@ YAML::Emitter &toYAML(YAML::Emitter &out, const Typelib::Numeric &type, const Ty
switch(type.getSize())
{
case sizeof(uint8_t):
out << *(static_cast<uint8_t *>(value.getData()));
//need to cast uint8_t to unsigned int so it is not interpreted as character but as number
out << static_cast<unsigned int>(*(static_cast<uint8_t *>(value.getData())));
break;
case sizeof(uint16_t):
out << *(static_cast<uint16_t *>(value.getData()));
Expand Down
4 changes: 2 additions & 2 deletions src/TypeWrapper.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ double TypeWrapper::toDouble() {
}
case Typelib::Numeric::SInt:
switch (numeric.getSize()) {
case sizeof(int8_t): return static_cast<int>(*static_cast<int8_t *>(value.getData()));
case sizeof(int8_t): return *static_cast<int8_t *>(value.getData());
case sizeof(int16_t): return *static_cast<int16_t*>(value.getData());
case sizeof(int32_t): return *static_cast<int32_t*>(value.getData());
case sizeof(int64_t): return *static_cast<int64_t*>(value.getData());
Expand All @@ -103,7 +103,7 @@ double TypeWrapper::toDouble() {
}
case Typelib::Numeric::UInt: {
switch (numeric.getSize()) {
case sizeof(uint8_t): return static_cast<unsigned int>(*static_cast<uint8_t *>(value.getData()));
case sizeof(uint8_t): return *static_cast<uint8_t *>(value.getData());
case sizeof(uint16_t): return *static_cast<uint16_t*>(value.getData());
case sizeof(uint32_t): return *static_cast<uint32_t*>(value.getData());
case sizeof(uint64_t): return *static_cast<uint64_t*>(value.getData());
Expand Down