diff options
author | falkTX <falktx@falktx.com> | 2022-12-10 23:21:07 +0000 |
---|---|---|
committer | falkTX <falktx@falktx.com> | 2022-12-10 23:21:07 +0000 |
commit | 52536b96c59982fb0ff7ae8055050a5d208bf2ed (patch) | |
tree | 76de13def9fe03619f4e960fe2236852af327a8a | |
parent | a2887f427bc444070d29ab30977a0caf475e50fc (diff) |
Fix potential name conflict with strncpy
Signed-off-by: falkTX <falktx@falktx.com>
-rw-r--r-- | distrho/src/DistrhoPluginCLAP.cpp | 22 | ||||
-rw-r--r-- | distrho/src/DistrhoPluginInternal.hpp | 2 | ||||
-rw-r--r-- | distrho/src/DistrhoPluginVST2.cpp | 36 | ||||
-rw-r--r-- | distrho/src/DistrhoPluginVST3.cpp | 32 |
4 files changed, 45 insertions, 47 deletions
diff --git a/distrho/src/DistrhoPluginCLAP.cpp b/distrho/src/DistrhoPluginCLAP.cpp index df7d37b8..1f642a01 100644 --- a/distrho/src/DistrhoPluginCLAP.cpp +++ b/distrho/src/DistrhoPluginCLAP.cpp @@ -1129,7 +1129,7 @@ public: if (hints & (kParameterIsBoolean|kParameterIsInteger)) info->flags |= CLAP_PARAM_IS_STEPPED; - DISTRHO_NAMESPACE::strncpy(info->name, fPlugin.getParameterName(index), CLAP_NAME_SIZE); + d_strncpy(info->name, fPlugin.getParameterName(index), CLAP_NAME_SIZE); uint wrtn; if (groupId != kPortGroupNone) @@ -1145,7 +1145,7 @@ public: wrtn = 0; } - DISTRHO_NAMESPACE::strncpy(info->module + wrtn, fPlugin.getParameterSymbol(index), CLAP_PATH_SIZE - wrtn); + d_strncpy(info->module + wrtn, fPlugin.getParameterSymbol(index), CLAP_PATH_SIZE - wrtn); } info->id = index; @@ -1182,7 +1182,7 @@ public: { if (d_isEqual(static_cast<double>(enumValues.values[i].value), value)) { - DISTRHO_NAMESPACE::strncpy(display, enumValues.values[i].label, size); + d_strncpy(display, enumValues.values[i].label, size); return true; } } @@ -1328,7 +1328,7 @@ public: const BusInfo& busInfo(busInfos[index]); info->id = busInfo.groupId; - DISTRHO_NAMESPACE::strncpy(info->name, busInfo.name, CLAP_NAME_SIZE); + d_strncpy(info->name, busInfo.name, CLAP_NAME_SIZE); info->flags = busInfo.isMain ? CLAP_AUDIO_PORT_IS_MAIN : 0x0; info->channel_count = busInfo.numChannels; @@ -1893,16 +1893,15 @@ private: case kPortGroupMono: if (busInfo.isMain) { - DISTRHO_NAMESPACE::strncpy(busInfo.name, - isInput ? "Audio Input" : "Audio Output", CLAP_NAME_SIZE); + d_strncpy(busInfo.name, isInput ? "Audio Input" : "Audio Output", CLAP_NAME_SIZE); break; } // fall-through default: if (group.name.isNotEmpty()) - DISTRHO_NAMESPACE::strncpy(busInfo.name, group.name, CLAP_NAME_SIZE); + d_strncpy(busInfo.name, group.name, CLAP_NAME_SIZE); else - DISTRHO_NAMESPACE::strncpy(busInfo.name, port.name, CLAP_NAME_SIZE); + d_strncpy(busInfo.name, port.name, CLAP_NAME_SIZE); break; } @@ -1935,7 +1934,7 @@ private: nonGroupSidechainId++ }; - DISTRHO_NAMESPACE::strncpy(busInfo.name, port.name, CLAP_NAME_SIZE); + d_strncpy(busInfo.name, port.name, CLAP_NAME_SIZE); busInfos.push_back(busInfo); } @@ -1961,12 +1960,11 @@ private: if (busInfo.isMain) { - DISTRHO_NAMESPACE::strncpy(busInfo.name, - isInput ? "Audio Input" : "Audio Output", CLAP_NAME_SIZE); + d_strncpy(busInfo.name, isInput ? "Audio Input" : "Audio Output", CLAP_NAME_SIZE); } else { - DISTRHO_NAMESPACE::strncpy(busInfo.name, port.name, CLAP_NAME_SIZE); + d_strncpy(busInfo.name, port.name, CLAP_NAME_SIZE); } busInfos.push_back(busInfo); diff --git a/distrho/src/DistrhoPluginInternal.hpp b/distrho/src/DistrhoPluginInternal.hpp index 413654d7..939be37b 100644 --- a/distrho/src/DistrhoPluginInternal.hpp +++ b/distrho/src/DistrhoPluginInternal.hpp @@ -89,7 +89,7 @@ void fillInPredefinedPortGroupData(const uint32_t groupId, PortGroup& portGroup) } static inline -void strncpy(char* const dst, const char* const src, const size_t length) +void d_strncpy(char* const dst, const char* const src, const size_t length) { DISTRHO_SAFE_ASSERT_RETURN(length > 0,); diff --git a/distrho/src/DistrhoPluginVST2.cpp b/distrho/src/DistrhoPluginVST2.cpp index 54ac83bf..b57366b8 100644 --- a/distrho/src/DistrhoPluginVST2.cpp +++ b/distrho/src/DistrhoPluginVST2.cpp @@ -484,7 +484,7 @@ public: case VST_EFFECT_OPCODE_04: // set program name if (char* const programName = (char*)ptr) { - strncpy(fProgramName, programName, 32); + d_strncpy(fProgramName, programName, 32); return 1; } break; @@ -492,7 +492,7 @@ public: case VST_EFFECT_OPCODE_05: // get program name if (char* const programName = (char*)ptr) { - strncpy(programName, fProgramName, 24); + d_strncpy(programName, fProgramName, 24); return 1; } break; @@ -500,7 +500,7 @@ public: case VST_EFFECT_OPCODE_1D: // get program name indexed if (char* const programName = (char*)ptr) { - strncpy(programName, fProgramName, 24); + d_strncpy(programName, fProgramName, 24); return 1; } break; @@ -1421,7 +1421,7 @@ static intptr_t VST_FUNCTION_INTERFACE vst_dispatcherCallback(vst_effect* const case VST_EFFECT_OPCODE_PARAM_GETLABEL: if (ptr != nullptr && index < static_cast<int32_t>(sPlugin->getParameterCount())) { - strncpy((char*)ptr, sPlugin->getParameterUnit(index), 8); + d_strncpy((char*)ptr, sPlugin->getParameterUnit(index), 8); return 1; } return 0; @@ -1431,9 +1431,9 @@ static intptr_t VST_FUNCTION_INTERFACE vst_dispatcherCallback(vst_effect* const { const String& shortName(sPlugin->getParameterShortName(index)); if (shortName.isNotEmpty()) - strncpy((char*)ptr, shortName, 16); + d_strncpy((char*)ptr, shortName, 16); else - strncpy((char*)ptr, sPlugin->getParameterName(index), 16); + d_strncpy((char*)ptr, sPlugin->getParameterName(index), 16); return 1; } return 0; @@ -1446,17 +1446,17 @@ static intptr_t VST_FUNCTION_INTERFACE vst_dispatcherCallback(vst_effect* const memset(properties, 0, sizeof(vst_parameter_properties)); // full name - strncpy(properties->name, - sPlugin->getParameterName(index), - sizeof(properties->name)); + d_strncpy(properties->name, + sPlugin->getParameterName(index), + sizeof(properties->name)); // short name const String& shortName(sPlugin->getParameterShortName(index)); if (shortName.isNotEmpty()) - strncpy(properties->label, - sPlugin->getParameterShortName(index), - sizeof(properties->label)); + d_strncpy(properties->label, + sPlugin->getParameterShortName(index), + sizeof(properties->label)); // parameter hints const uint32_t hints = sPlugin->getParameterHints(index); @@ -1496,9 +1496,9 @@ static intptr_t VST_FUNCTION_INTERFACE vst_dispatcherCallback(vst_effect* const { properties->flags |= VST_PARAMETER_FLAGS_CATEGORY; properties->category = i + 1; - strncpy(properties->category_label, - portGroup.name.buffer(), - sizeof(properties->category_label)); + d_strncpy(properties->category_label, + portGroup.name.buffer(), + sizeof(properties->category_label)); break; } } @@ -1526,7 +1526,7 @@ static intptr_t VST_FUNCTION_INTERFACE vst_dispatcherCallback(vst_effect* const case VST_EFFECT_OPCODE_EFFECT_NAME: if (char* const cptr = (char*)ptr) { - strncpy(cptr, sPlugin->getName(), 32); + d_strncpy(cptr, sPlugin->getName(), 32); return 1; } return 0; @@ -1534,7 +1534,7 @@ static intptr_t VST_FUNCTION_INTERFACE vst_dispatcherCallback(vst_effect* const case VST_EFFECT_OPCODE_VENDOR_NAME: if (char* const cptr = (char*)ptr) { - strncpy(cptr, sPlugin->getMaker(), 32); + d_strncpy(cptr, sPlugin->getMaker(), 32); return 1; } return 0; @@ -1542,7 +1542,7 @@ static intptr_t VST_FUNCTION_INTERFACE vst_dispatcherCallback(vst_effect* const case VST_EFFECT_OPCODE_PRODUCT_NAME: if (char* const cptr = (char*)ptr) { - strncpy(cptr, sPlugin->getLabel(), 32); + d_strncpy(cptr, sPlugin->getLabel(), 32); return 1; } return 0; diff --git a/distrho/src/DistrhoPluginVST3.cpp b/distrho/src/DistrhoPluginVST3.cpp index 647a500b..ed12ebc0 100644 --- a/distrho/src/DistrhoPluginVST3.cpp +++ b/distrho/src/DistrhoPluginVST3.cpp @@ -4715,9 +4715,9 @@ struct dpf_factory : v3_plugin_factory_cpp { std::memset(info, 0, sizeof(*info)); info->flags = 0x10; // unicode - DISTRHO_NAMESPACE::strncpy(info->vendor, sPlugin->getMaker(), ARRAY_SIZE(info->vendor)); - DISTRHO_NAMESPACE::strncpy(info->url, sPlugin->getHomePage(), ARRAY_SIZE(info->url)); - // DISTRHO_NAMESPACE::strncpy(info->email, "", ARRAY_SIZE(info->email)); // TODO + d_strncpy(info->vendor, sPlugin->getMaker(), ARRAY_SIZE(info->vendor)); + d_strncpy(info->url, sPlugin->getHomePage(), ARRAY_SIZE(info->url)); + // d_strncpy(info->email, "", ARRAY_SIZE(info->email)); // TODO return V3_OK; } @@ -4738,17 +4738,17 @@ struct dpf_factory : v3_plugin_factory_cpp { DISTRHO_SAFE_ASSERT_RETURN(idx <= 2, V3_INVALID_ARG); info->cardinality = 0x7FFFFFFF; - DISTRHO_NAMESPACE::strncpy(info->name, sPlugin->getName(), ARRAY_SIZE(info->name)); + d_strncpy(info->name, sPlugin->getName(), ARRAY_SIZE(info->name)); if (idx == 0) { std::memcpy(info->class_id, dpf_tuid_class, sizeof(v3_tuid)); - DISTRHO_NAMESPACE::strncpy(info->category, "Audio Module Class", ARRAY_SIZE(info->category)); + d_strncpy(info->category, "Audio Module Class", ARRAY_SIZE(info->category)); } else { std::memcpy(info->class_id, dpf_tuid_controller, sizeof(v3_tuid)); - DISTRHO_NAMESPACE::strncpy(info->category, "Component Controller Class", ARRAY_SIZE(info->category)); + d_strncpy(info->category, "Component Controller Class", ARRAY_SIZE(info->category)); } return V3_OK; @@ -4806,21 +4806,21 @@ struct dpf_factory : v3_plugin_factory_cpp { #if DPF_VST3_USES_SEPARATE_CONTROLLER || !DISTRHO_PLUGIN_HAS_UI info->class_flags = V3_DISTRIBUTABLE; #endif - DISTRHO_NAMESPACE::strncpy(info->sub_categories, getPluginCategories(), ARRAY_SIZE(info->sub_categories)); - DISTRHO_NAMESPACE::strncpy(info->name, sPlugin->getName(), ARRAY_SIZE(info->name)); - DISTRHO_NAMESPACE::strncpy(info->vendor, sPlugin->getMaker(), ARRAY_SIZE(info->vendor)); - DISTRHO_NAMESPACE::strncpy(info->version, getPluginVersion(), ARRAY_SIZE(info->version)); - DISTRHO_NAMESPACE::strncpy(info->sdk_version, "Travesty 3.7.4", ARRAY_SIZE(info->sdk_version)); + d_strncpy(info->sub_categories, getPluginCategories(), ARRAY_SIZE(info->sub_categories)); + d_strncpy(info->name, sPlugin->getName(), ARRAY_SIZE(info->name)); + d_strncpy(info->vendor, sPlugin->getMaker(), ARRAY_SIZE(info->vendor)); + d_strncpy(info->version, getPluginVersion(), ARRAY_SIZE(info->version)); + d_strncpy(info->sdk_version, "Travesty 3.7.4", ARRAY_SIZE(info->sdk_version)); if (idx == 0) { std::memcpy(info->class_id, dpf_tuid_class, sizeof(v3_tuid)); - DISTRHO_NAMESPACE::strncpy(info->category, "Audio Module Class", ARRAY_SIZE(info->category)); + d_strncpy(info->category, "Audio Module Class", ARRAY_SIZE(info->category)); } else { std::memcpy(info->class_id, dpf_tuid_controller, sizeof(v3_tuid)); - DISTRHO_NAMESPACE::strncpy(info->category, "Component Controller Class", ARRAY_SIZE(info->category)); + d_strncpy(info->category, "Component Controller Class", ARRAY_SIZE(info->category)); } return V3_OK; @@ -4839,7 +4839,7 @@ struct dpf_factory : v3_plugin_factory_cpp { #if DPF_VST3_USES_SEPARATE_CONTROLLER || !DISTRHO_PLUGIN_HAS_UI info->class_flags = V3_DISTRIBUTABLE; #endif - DISTRHO_NAMESPACE::strncpy(info->sub_categories, getPluginCategories(), ARRAY_SIZE(info->sub_categories)); + d_strncpy(info->sub_categories, getPluginCategories(), ARRAY_SIZE(info->sub_categories)); DISTRHO_NAMESPACE::strncpy_utf16(info->name, sPlugin->getName(), ARRAY_SIZE(info->name)); DISTRHO_NAMESPACE::strncpy_utf16(info->vendor, sPlugin->getMaker(), ARRAY_SIZE(info->vendor)); DISTRHO_NAMESPACE::strncpy_utf16(info->version, getPluginVersion(), ARRAY_SIZE(info->version)); @@ -4848,12 +4848,12 @@ struct dpf_factory : v3_plugin_factory_cpp { if (idx == 0) { std::memcpy(info->class_id, dpf_tuid_class, sizeof(v3_tuid)); - DISTRHO_NAMESPACE::strncpy(info->category, "Audio Module Class", ARRAY_SIZE(info->category)); + d_strncpy(info->category, "Audio Module Class", ARRAY_SIZE(info->category)); } else { std::memcpy(info->class_id, dpf_tuid_controller, sizeof(v3_tuid)); - DISTRHO_NAMESPACE::strncpy(info->category, "Component Controller Class", ARRAY_SIZE(info->category)); + d_strncpy(info->category, "Component Controller Class", ARRAY_SIZE(info->category)); } return V3_OK; |