Skip to content

[poc]Cpesp 7136 extern #143

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

Open
santoshcomcast wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

[poc]Cpesp 7136 extern #143

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
bec9c03
POC: read the variable through dlsym utlity.
santoshcomcast Sep 10, 2025
a1ce0dd
read variable through dlsym utility
santoshcomcast Sep 10, 2025
34ef765
debug extern from lib
santoshcomcast Sep 10, 2025
7deaac8
debug poc
santoshcomcast Sep 11, 2025
adf06cc
debug poc
santoshcomcast Sep 11, 2025
eb49b4a
debug poc
santoshcomcast Sep 12, 2025
1de2bef
debug poc
santoshcomcast Sep 12, 2025
4e473dd
debug build issue
santoshcomcast Sep 15, 2025
1211fad
debug poc
santoshcomcast Sep 15, 2025
0521d96
debug poc
santoshcomcast Sep 16, 2025
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
2 changes: 1 addition & 1 deletion ds/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ all: install

library: $(OBJS)
@echo "Building $(LIBNAMEFULL) ...."
$(CXX) $(OBJS) $(CFLAGS) $(DSHAL_LDFLAGS) -L$(INSTALL)/lib -ldshalcli -shared -o $(LIBNAMEFULL)
$(CXX) $(OBJS) $(CFLAGS) $(DSHAL_LDFLAGS) -L$(INSTALL)/lib -ldshalcli -ldshalsrv -shared -o $(LIBNAMEFULL)
$(CXX) $(OBJS) $(CFLAGS) -L$(INSTALL)/lib -ldshalcli -shared -o $(LIBNAMECLI)

%.o: %.cpp
Expand Down
226 changes: 225 additions & 1 deletion ds/audioOutputPortConfig.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/*
* If not stated otherwise in this file or this component's LICENSE file the
* following copyright and licenses apply:

Check failure on line 3 in ds/audioOutputPortConfig.cpp

View workflow job for this annotation

GitHub Actions / call-fossid-workflow / Fossid Annotate PR

FossID License Issue Detected 

Source code with 'Apache-2.0' license found in local file 'ds/audioOutputPortConfig.cpp' (Match: rdk/components/generic/devicesettings/rdk/components/generic/devicesettings/2.1-20161031, 415 lines, url: https://code.rdkcentral.com/r/plugins/gitiles/rdk/components/generic/devicesettings/+archive/2.1-20161031.tar.gz, file: ds/audioOutputPortConfig.cpp)
*
* Copyright 2016 RDK Management
*
Expand Down Expand Up @@ -35,6 +35,23 @@
#include "stdlib.h"
#include "dslogger.h"

#include <dlfcn.h>
#include "libIARM.h"
#include "libIBus.h"
#include "iarmUtil.h"

#define IARM_BUS_Lock(lock) pthread_mutex_lock(&dsLock)
#define IARM_BUS_Unlock(lock) pthread_mutex_unlock(&dsLock)
extern dsAudioTypeConfig_t kConfig_audio[dsAUDIOPORT_TYPE_MAX];
extern dsAudioPortConfig_t kPort_audio[dsAUDIOPORT_TYPE_MAX];
extern int kConfig_size, kPort_size;
static dsAudioTypeConfig_t *kConfigs1 = NULL;//, *kConfigs2 = kConfig_audio;
static dsAudioPortConfig_t *kPorts1 = NULL;//, *kPorts2 = kPort_audio;
int *pKConSize, *pKPortSize;

static pthread_mutex_t dsLock = PTHREAD_MUTEX_INITIALIZER;


namespace device {

//To Make the instance as thread-safe, using = default, that can request special methods from the compiler. They are Special because only compiler can create them.
Expand Down Expand Up @@ -108,6 +125,164 @@
return supportedTypes;
}

#if 1
bool searchConfigs()
{
//static dsAudioTypeConfig_t *kConfigs1 = NULL;
//static dsAudioPortConfig_t *kPorts1 = NULL;
int kConfig_size_local = -1, kPort_size_local = -1;
int kConfig_size_dl = -1, kPort_size_dl = -1;
//IARM_BUS_Lock(lock);
//if ( kConfigs1 == NULL)
{
void *dllib = dlopen("libdshalsrv.so", RTLD_LAZY);
if (dllib) {
kConfigs1 = (dsAudioTypeConfig_t *) dlsym(dllib, "kConfig_audio");
if (kConfigs1) {
INT_DEBUG("kConfig_audio is defined and loaded kConfigs1 = %p\r\n", kConfigs1);
printf("%d:%s: kConfig_audio is defined and loaded kConfigs1 = %p\n", __LINE__, __func__, kConfigs1);
}
else {
INT_INFO("kConfig_audio is not defined\r\n");
printf("%d:%s: kConfig_audio is not defined\n", __LINE__, __func__);
//IARM_BUS_Unlock(lock);
//dlclose(dllib);
//return IARM_RESULT_INVALID_STATE;
}

pKConSize = (int *) dlsym(dllib, "kConfig_size");
pKPortSize = (int *) dlsym(dllib, "kPort_size");
if(pKConSize)
{
kConfig_size_local = *pKConSize;
printf("%d:%s: pKConSize is defined and loaded kConfig_size_local = %d\n", __LINE__, __func__, kConfig_size_local);
}
else
{
printf("%d:%s: pKConSize is not defined\n", __LINE__, __func__);
kConfig_size_local = -1;
}
if(pKPortSize)
{
kPort_size_local = *pKPortSize;
printf("%d:%s: pKPortSize is defined and loaded kPort_size_local = %d\n", __LINE__, __func__, kPort_size_local);
}
else
{
printf("%d:%s: pKPortSize is not defined\n", __LINE__, __func__);
kPort_size_local = -1;
}
dlclose(dllib);
}
else {
INT_ERROR("Opening libdshalsrv.so failed\r\n");
printf("%d:%s: Opening libdshalsrv.so failed\n", __LINE__, __func__);
}
}

//if ( kPorts1 == NULL)
{
void *dllib = dlopen("libdshalsrv.so", RTLD_LAZY);
if (dllib) {
kPorts1 = (dsAudioPortConfig_t *) dlsym(dllib, "kPort_audio");
if (kPorts1) {
INT_DEBUG("kPort_audio is defined and loaded kPorts1 = %p\r\n", kPorts1);
printf("%d:%s: kPort_audio is defined and loaded kPorts1 = %p\n", __LINE__, __func__, kPorts1);
}
else {
INT_INFO("kPort_audio is not defined\r\n");
printf("%d:%s: kPort_audio is not defined\n", __LINE__, __func__);
//IARM_BUS_Unlock(lock);
//dlclose(dllib);
//return IARM_RESULT_INVALID_STATE;
}
dlclose(dllib);
}
else {
INT_ERROR("Opening libdshalsrv.so failed\r\n");
printf("%d:%s: Opening libdshalsrv.so failed\n", __LINE__, __func__);
}
}
//dlclose(dllib);
//IARM_BUS_Unlock(lock);
//kConfig_size_dl = dsUTL_DIM(kConfigs1);
//kPort_size_dl = dsUTL_DIM(kPorts1);
//printf("%d:%s: dsUTL_DIM kConfig_size_dl = %d kPort_size_dl = %d\n", __LINE__, __func__, kConfig_size_dl, kPort_size_dl);
#if 0
printf("\n\n=========================================================================================================================\n\n");
printf("\n%d:%s print configs dlsym\n", __LINE__, __func__);
if
for (size_t i = 0; i < kConfig_size_local; i++) {
Copy link
Contributor

@rdkcmf-jenkins rdkcmf-jenkins Sep 11, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverity issue no longer present as of: undefined

Show issue

Coverity Issue - Negative loop bound

Using unsigned variable "kConfig_size_local" in a loop exit condition.

Medium Impact, CWE-606
NEGATIVE_RETURNS

//const dsAudioTypeConfig_t *typeCfg = &kConfig_audio[i];
const dsAudioTypeConfig_t *typeCfg = &kConfigs1[i]; //NO compile issues
AudioOutputPortType &aPortType = AudioOutputPortType::getInstance(typeCfg->typeId);
aPortType.enable();
printf("%d:%s: typeCfg->typeId = %d\n", __LINE__, __func__, typeCfg->typeId);
printf("%d:%s: typeCfg->name = %s\n", __LINE__, __func__, typeCfg->name);
printf("%d:%s: typeCfg->numSupportedEncodings = %zu\n", __LINE__, __func__, typeCfg->numSupportedEncodings);
printf("%d:%s: typeCfg->numSupportedCompressions = %zu\n", __LINE__, __func__, typeCfg->numSupportedCompressions);
printf("%d:%s: typeCfg->numSupportedStereoModes = %zu\n", __LINE__, __func__, typeCfg->numSupportedStereoModes);
}


/*
* set up ports based on kPorts[]
*/
for (size_t i = 0; i < kPort_size_local; i++) {
Copy link
Contributor

@rdkcmf-jenkins rdkcmf-jenkins Sep 11, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverity issue no longer present as of: undefined

Show issue

Coverity Issue - Negative loop bound

Using unsigned variable "kPort_size_local" in a loop exit condition.

Medium Impact, CWE-606
NEGATIVE_RETURNS

//const dsAudioPortConfig_t *port = &kPort_audio[i];
const dsAudioPortConfig_t *port = &kPorts1[i];
printf("%d:%s: port->id.type = %d\n", __LINE__, __func__, port->id.type);
printf("%d:%s: port->id.index = %d\n", __LINE__, __func__, port->id.index);
}
printf("\n\n=========================================================================================================================\n\n");
#else
printf("\n\n=========================================================================================================================\n\n");
printf("\n%d:%s print configs using extern\n", __LINE__, __func__);
//printf("%d:%s: size of(kConfig_audio) = %d size of(kPort_audio) = %d\n", __LINE__, __func__, kConfig_size, kPort_size);
//kConfig_size_local = kConfig_size;
//kPort_size_local = kPort_size;

if(kConfigs1 != NULL)
{
for (size_t i = 0; i < kConfig_size_local; i++) {
Copy link
Contributor

@rdkcmf-jenkins rdkcmf-jenkins Sep 12, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverity issue no longer present as of: undefined

Show issue

Coverity Issue - Negative loop bound

Using unsigned variable "kConfig_size_local" in a loop exit condition.

Medium Impact, CWE-606
NEGATIVE_RETURNS

//const dsAudioTypeConfig_t *typeCfg = &kConfig_audio[i];
const dsAudioTypeConfig_t *typeCfg = &kConfigs1[i];
AudioOutputPortType &aPortType = AudioOutputPortType::getInstance(typeCfg->typeId);
aPortType.enable();
printf("%d:%s: typeCfg->typeId = %d\n", __LINE__, __func__, typeCfg->typeId);
printf("%d:%s: typeCfg->name = %s\n", __LINE__, __func__, typeCfg->name);
printf("%d:%s: typeCfg->numSupportedEncodings = %zu\n", __LINE__, __func__, typeCfg->numSupportedEncodings);
printf("%d:%s: typeCfg->numSupportedCompressions = %zu\n", __LINE__, __func__, typeCfg->numSupportedCompressions);
printf("%d:%s: typeCfg->numSupportedStereoModes = %zu\n", __LINE__, __func__, typeCfg->numSupportedStereoModes);
}
}
else
{
printf("%d:%s: kConfig_audio is NULL\n", __LINE__, __func__);
}
if(kPorts1 != NULL)
{
/*
* set up ports based on kPorts[]
*/
for (size_t i = 0; i < kPort_size_local; i++) {
Copy link
Contributor

@rdkcmf-jenkins rdkcmf-jenkins Sep 12, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverity issue no longer present as of: undefined

Show issue

Coverity Issue - Negative loop bound

Using unsigned variable "kPort_size_local" in a loop exit condition.

Medium Impact, CWE-606
NEGATIVE_RETURNS

//const dsAudioPortConfig_t *port = &kPort_audio[i];
const dsAudioPortConfig_t *port = &kPorts1[i];
printf("%d:%s: port->id.type = %d\n", __LINE__, __func__, port->id.type);
printf("%d:%s: port->id.index = %d\n", __LINE__, __func__, port->id.index);
}
}
else
{
printf("%d:%s: kPort_audio is NULL\n", __LINE__, __func__);
}
printf("\n\n=========================================================================================================================\n\n");
#endif

return (kConfigs1 || kPorts1);
}
#endif

void AudioOutputPortConfig::load()
{
try {
Expand All @@ -132,7 +307,7 @@
_aPortTypes.push_back(AudioOutputPortType(i));

}

#if 1
/*
* Initialize Audio portTypes (encodings, compressions etc.)
* and its port instances (db, level etc)
Expand All @@ -157,6 +332,7 @@
}
}


/*
* set up ports based on kPorts[]
*/
Expand All @@ -165,6 +341,54 @@
_aPorts.push_back(AudioOutputPort((port->id.type), port->id.index, i));
_aPortTypes.at(port->id.type).addPort(_aPorts.at(i));
}
searchConfigs();
#else //gsk
if(searchConfigs())
{
/*if(kConfig_audio == NULL || kPort_audio == NULL) {
throw IllegalArgumentException();
}*/
//dsAudioTypeConfig_t *kConfigs = kConfig_audio;
//dsAudioPortConfig_t *kPorts = kPort_audio;
/*
* Initialize Audio portTypes (encodings, compressions etc.)
* and its port instances (db, level etc)
*/
for (size_t i = 0; i < dsUTL_DIM(kConfigs1); i++) {
const dsAudioTypeConfig_t *typeCfg = &kConfigs1[i];
AudioOutputPortType &aPortType = AudioOutputPortType::getInstance(typeCfg->typeId);
aPortType.enable();
for (size_t j = 0; j < typeCfg->numSupportedEncodings; j++) {
aPortType.addEncoding(AudioEncoding::getInstance(typeCfg->encodings[j]));
_aEncodings.at(typeCfg->encodings[j]).enable();
}
for (size_t j = 0; j < typeCfg->numSupportedCompressions; j++) {
aPortType.addCompression(typeCfg->compressions[j]);
_aCompressions.at(typeCfg->compressions[j]).enable();

}
for (size_t j = 0; j < typeCfg->numSupportedStereoModes; j++) {
aPortType.addStereoMode(typeCfg->stereoModes[j]);
_aStereoModes.at(typeCfg->stereoModes[j]).enable();

}
}


/*
* set up ports based on kPorts[]
*/
for (size_t i = 0; i < dsUTL_DIM(kPorts1); i++) {
const dsAudioPortConfig_t *port = &kPorts1[i];
_aPorts.push_back(AudioOutputPort((port->id.type), port->id.index, i));
_aPortTypes.at(port->id.type).addPort(_aPorts.at(i));
}
}
else
{
printf("%d:%s: kConfig and kPorts are not available\n", __LINE__, __func__);
}
#endif

}
catch(const Exception &e) {
Expand Down
Loading
Loading