Skip to content

U
uniset2
Commit 29709852 authored by Pavel Vainerman's avatar Pavel Vainerman
Browse files
Options

Coverity scan:

[fixed]: Uninitialized variables (UNINIT) [fixed]: Error handling issues (UNCAUGHT_EXCEPT) [fixed]: Control flow issues (NO_EFFECT) [fixed]: Uninitialized scalar variable (UNINIT)
parent a7d8e78a
Show whitespace changes
Inline Side-by-side
Showing with 183 additions and 136 deletions
Utilities/MBTester/mbrtutester.cc
View file @ 29709852
......@@ -479,7 +479,7 @@ int main( int argc, char** argv )
cout << "(reply): count=" << ModbusRTU::dat2str(ret.count) << endl;
for( int i = 0; i < ret.count; i++ )
for( size_t i = 0; i < ret.count; i++ )
{
cout << i << ": (" << ModbusRTU::dat2str( reg + i ) << ") = " << (int)(ret.data[i])
<< " ("
......@@ -507,7 +507,7 @@ int main( int argc, char** argv )
cout << "(reply): count=" << ModbusRTU::dat2str(ret.count) << endl;
for( int i = 0; i < ret.count; i++ )
for( size_t i = 0; i < ret.count; i++ )
{
cout << i << ": (" << ModbusRTU::dat2str( reg + i ) << ") = " << (int)(ret.data[i])
<< " ("
......
Utilities/MBTester/mbtcptester.cc
View file @ 29709852
......@@ -421,7 +421,7 @@ int main( int argc, char** argv )
cout << "(reply): count=" << ModbusRTU::dat2str(ret.count) << endl;
for( int i = 0; i < ret.count; i++ )
for( size_t i = 0; i < ret.count; i++ )
{
cout << i << ": (" << ModbusRTU::dat2str( reg + i ) << ") = " << (int)(ret.data[i])
<< " ("
......@@ -449,7 +449,7 @@ int main( int argc, char** argv )
cout << "(reply): count=" << ModbusRTU::dat2str(ret.count) << endl;
for( int i = 0; i < ret.count; i++ )
for( size_t i = 0; i < ret.count; i++ )
{
cout << i << ": (" << ModbusRTU::dat2str( reg + i ) << ") = " << (int)(ret.data[i])
<< " ("
......
Utilities/ULog/log-stdin.cc
View file @ 29709852
......@@ -79,7 +79,7 @@ int main( int argc, char* argv[], char* envp[] )
while( true )
{
size_t r = read(fileno(stdin), buf, sizeof(buf) - 1);
ssize_t r = read(fileno(stdin), buf, sizeof(buf) - 1);
if( r > 0 )
{
......
Utilities/codegen/ctl-cpp-common.xsl
View file @ 29709852
......@@ -858,11 +858,13 @@ end_private(false)
resetMsgTime = uni_atoi(init3_str(conf->getArgParam("--" + argprefix + "resetMsgTime"),conf->getProp(cnode,"resetMsgTime"),s_resetTime));
ptResetMsg.setTiming(resetMsgTime);
smReadyTimeout = conf->getArgInt("--" + argprefix + "sm-ready-timeout","<xsl:call-template name="settings"><xsl:with-param name="varname" select="'smReadyTimeout'"/></xsl:call-template>");
if( smReadyTimeout == 0 )
int sm_tout = conf->getArgInt("--" + argprefix + "sm-ready-timeout","<xsl:call-template name="settings"><xsl:with-param name="varname" select="'smReadyTimeout'"/></xsl:call-template>");
if( sm_tout == 0 )
smReadyTimeout = 60000;
else if( smReadyTimeout &lt; 0 )
else if( sm_tout &lt; 0 )
smReadyTimeout = UniSetTimer::WaitUpTime;
else
smReadyTimeout = sm_tout;
smTestID = conf->getSensorID(init3_str(conf->getArgParam("--" + argprefix + "sm-test-id"),conf->getProp(cnode,"smTestID"),""));
<xsl:for-each select="//smap/item">
......@@ -1267,11 +1269,13 @@ askPause(uniset_conf()->getPIntProp(cnode,"askPause",2000))
resetMsgTime = uni_atoi(init3_str(conf->getArgParam("--" + argprefix + "resetMsgTime"),conf->getProp(cnode,"resetMsgTime"),s_resetTime));
ptResetMsg.setTiming(resetMsgTime);
smReadyTimeout = conf->getArgInt("--" + argprefix + "sm-ready-timeout","<xsl:call-template name="settings"><xsl:with-param name="varname" select="'smReadyTimeout'"/></xsl:call-template>");
if( smReadyTimeout == 0 )
int sm_tout = conf->getArgInt("--" + argprefix + "sm-ready-timeout","<xsl:call-template name="settings"><xsl:with-param name="varname" select="'smReadyTimeout'"/></xsl:call-template>");
if( sm_tout == 0 )
smReadyTimeout = 60000;
else if( smReadyTimeout &lt; 0 )
else if( sm_tout &lt; 0 )
smReadyTimeout = UniSetTimer::WaitUpTime;
else
smReadyTimeout = sm_tout;
smTestID = conf->getSensorID(init3_str(conf->getArgParam("--" + argprefix + "sm-test-id"),conf->getProp(cnode,"smTestID"),""));
......@@ -1454,11 +1458,7 @@ std::string <xsl:value-of select="$CLASSNAME"/>_SK::strval( UniSetTypes::Object
{
<xsl:if test="normalize-space($onlymsg)=''">
</xsl:if>
// приходится искуственно использовать третий параметр,
// что-бы компилятор выбрал
// правильный(для аналоговых) конструктор у SensorMessage
IOController_i::CalibrateInfo _ci;
SensorMessage _sm( <xsl:value-of select="@name"/>, (long)<xsl:call-template name="setprefix"/><xsl:value-of select="@name"/>, _ci);
SensorMessage _sm( <xsl:value-of select="@name"/>, (long)<xsl:call-template name="setprefix"/><xsl:value-of select="@name"/> );
_sm.sensor_type = UniversalIO::AI;
sensorInfo(&amp;_sm);
}
......
extensions/DBServer-PostgreSQL/DBServer_PostgreSQL.cc
View file @ 29709852
......@@ -303,7 +303,10 @@ void DBServer_PostgreSQL::createTables( std::shared_ptr<PostgreSQLInterface>& db
throw Exception();
}
for( it.goChildren(); it; it.goNext() )
if( !it.goChildren() )
return;
for( ; it; it.goNext() )
{
if( it.getName() != "comment" )
{
......
extensions/DBServer-PostgreSQL/PostgreSQLInterface.cc
View file @ 29709852
......@@ -203,11 +203,11 @@ PostgreSQLResult::~PostgreSQLResult()
// -----------------------------------------------------------------------------------------
PostgreSQLResult::PostgreSQLResult( const pqxx::result& res )
{
for (result::const_iterator c = res.begin(); c != res.end(); ++c)
for( result::const_iterator c = res.begin(); c != res.end(); ++c )
{
COL col;
for( pqxx::result::tuple::size_type i = 0; i < c.size(); i++ )
for( auto i = 0; i < c.size(); i++ )
col.push_back( c[i].as<string>() );
row.push_back(col);
......
extensions/DBServer-SQLite/DBServer_SQLite.cc
View file @ 29709852
......@@ -306,7 +306,10 @@ void DBServer_SQLite::createTables( SQLiteInterface* db )
throw Exception();
}
for( it.goChildren(); it; it.goNext() )
if( !it.goChildren() )
return;
for( ; it; it.goNext() )
{
if( it.getName() != "comment" )
{
......
extensions/IOControl/IOControl.cc
View file @ 29709852
......@@ -1569,6 +1569,8 @@ void IOControl::buildCardsList()
UniXML::iterator it1(nnode);
it1.goChildren();
if( it1.goChildren() )
{
for( ; it1.getCurrent(); it1.goNext() )
{
if( it1.getProp("name") == conf->getLocalNodeName() )
......@@ -1577,6 +1579,7 @@ void IOControl::buildCardsList()
break;
}
}
}
if( !mynode )
{
......
extensions/IOControl/iocalibr.cc
View file @ 29709852
......@@ -188,7 +188,13 @@ int main(int argc, char* argv[])
cout << "\r" << "data: " << setw(5) << data << " " << flush;
int temp = read(fd, &buf, sizeof(buf));
ssize_t temp = read(fd, &buf, sizeof(buf));
if( temp == -1 )
{
close(fd);
cerr << "can't read from stdin.. error: " << strerror(errno) << endl;
exit(EXIT_FAILURE);
}
if(temp == 1)
readCalibr(data);
......@@ -319,9 +325,21 @@ void readCalibr(int fixed)
cout << " Enter calibrated value for data=" << fixed << "\n cal = " << flush;
int f = open("/dev/stdin", O_RDONLY );
if( f == -1 )
{
cerr << "read calibrate fail.. error: " << strerror(errno) << endl;
return;
}
memset(rbuf, 0, sizeof(rbuf));
int temp = read(f, &rbuf, sizeof(rbuf));
ssize_t temp = read(f, &rbuf, sizeof(rbuf));
if( temp == -1 )
{
cerr << "read calibrate fail.. error: " << strerror(errno) << endl;
close(f);
return;
}
if (temp > 1)
{
int s;
......@@ -336,6 +354,8 @@ void readCalibr(int fixed)
}
else
cout << " you must input only any digits!" << endl;
close(f);
}
// --------------------------------------------------------------------------
......@@ -533,7 +553,7 @@ void sortData(bool rise, bool cal)
list<int>::iterator ite;
int tt = *(temp.begin());
for(itl = temp.end(); itl != temp.begin(); --itl)
for(itl = --temp.end(); itl != temp.begin(); --itl)
{
if(tt == *itl)continue;
......
extensions/LogicProcessor/tests/tests.cc
View file @ 29709852
......@@ -12,6 +12,8 @@ using namespace UniSetTypes;
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -27,8 +29,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
auto conf = uniset_init(argc, argv, "lp-configure.xml");
auto act = UniSetActivator::Instance();
......
extensions/ModbusMaster/RTUStorage.cc
View file @ 29709852
......@@ -130,7 +130,11 @@ void RTUStorage::poll( const std::shared_ptr<ModbusRTUMaster>& mb ) throw( Modbu
if( ret.getData(b, bits) )
{
for( unsigned int i = 0; i < 8; i++ )
unio_do[i + 8 * b] = bits[i];
{
int k = i + 8 * b;
if( k<48 )
unio_do[k] = bits[i];
}
}
}
}
......@@ -145,7 +149,11 @@ void RTUStorage::poll( const std::shared_ptr<ModbusRTUMaster>& mb ) throw( Modbu
if( ret.getData(b, bits) )
{
for( unsigned int i = 0; i < 8; i++ )
unio_di[i + 8 * b] = bits[i];
{
int k = i + 8 * b;
if( k<48 )
unio_di[k] = bits[i];
}
}
}
}
......
extensions/ModbusMaster/tests/run_test_mbtcpmaster.cc
View file @ 29709852
......@@ -18,6 +18,8 @@ std::shared_ptr<SharedMemory> shm;
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -34,8 +36,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
auto conf = uniset_init(argc, argv);
dlog()->logFile("./smtest.log");
......
extensions/ModbusMaster/tests/run_test_mbtcpmultimaster.cc
View file @ 29709852
......@@ -15,6 +15,8 @@ using namespace UniSetExtensions;
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -31,8 +33,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
auto conf = uniset_init(argc, argv);
dlog()->logFile("./smtest.log");
......
extensions/ModbusSlave/tests/tests_with_sm.cc
View file @ 29709852
......@@ -15,6 +15,8 @@ using namespace UniSetExtensions;
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -31,8 +33,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
auto conf = uniset_init(argc, argv);
bool apart = findArgParam("--apart", argc, argv) != -1;
......
extensions/SharedMemory/tests/Makefile.am
View file @ 29709852
......@@ -2,7 +2,7 @@ if HAVE_TESTS
noinst_PROGRAMS = tests
tests_SOURCES = NullSM.cc TestObject_SK.cc TestObject.cc tests.cc test_sm.cc
tests_SOURCES = NullSM.cc TestObject_SK.h TestObject_SK.cc TestObject.cc tests.cc test_sm.cc
tests_LDADD = $(top_builddir)/lib/libUniSet2.la $(top_builddir)/extensions/lib/libUniSet2Extensions.la \
$(top_builddir)/extensions/SharedMemory/libUniSet2SharedMemory.la \
$(SIGC_LIBS) $(COMCPP_LIBS)
......
extensions/SharedMemory/tests/tests.cc
View file @ 29709852
......@@ -18,6 +18,8 @@ std::shared_ptr<TestObject> obj;
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -34,8 +36,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
auto conf = uniset_init(argc, argv);
auto shm = SharedMemory::init_smemory(argc, argv);
......
extensions/UNetUDP/UDPPacket.h
View file @ 29709852
......@@ -28,6 +28,8 @@ namespace UniSetUDP
unsigned long num;
long nodeID;
long procID;
//!\todo может лучше использовать системно-независимый unsigned long, чем size_t?
size_t dcount; /*!< количество булевых величин */
size_t acount; /*!< количество аналоговых величин */
......@@ -35,7 +37,7 @@ namespace UniSetUDP
friend std::ostream& operator<<( std::ostream& os, UDPHeader* p );
} __attribute__((packed));
const unsigned long MaxPacketNum = std::numeric_limits<unsigned long>::max();
const size_t MaxPacketNum = std::numeric_limits<size_t>::max();
struct UDPAData
{
......@@ -100,18 +102,18 @@ namespace UniSetUDP
{
return !((dcount < MaxDCount) && (acount < MaxACount));
}
inline int dsize() const
inline size_t dsize() const
{
return dcount;
}
inline int asize() const
inline size_t asize() const
{
return acount;
}
unsigned short getDataCRC();
// количество байт в пакете с булевыми переменными...
int d_byte() const
size_t d_byte() const
{
return dcount * sizeof(long) + dcount;
}
......
extensions/UNetUDP/tests/test_unetudp.cc
View file @ 29709852
......@@ -158,12 +158,14 @@ TEST_CASE("[UNetUDP]: UDPMessage", "[unetudp][udpmessage]")
{
// создаём сообщение, преобразуем к Package.. потом обратно.. проверяём, что информация не исказилась
UniSetUDP::UDPMessage u;
int a = u.addAData(100, 100);
int d = u.addDData(110, true);
size_t a = u.addAData(100, 100);
size_t d = u.addDData(110, true);
UniSetUDP::UDPPacket p;
size_t len = u.transport_msg(p);
CHECK( len != 0 );
CHECK( a < UniSetUDP::MaxACount );
CHECK( a < UniSetUDP::MaxDCount );
UniSetUDP::UDPMessage u2(p);
REQUIRE( u2.a_dat[a].id == 100 );
......@@ -194,7 +196,7 @@ TEST_CASE("[UNetUDP]: check sender", "[unetudp][sender]")
REQUIRE( pack.asize() == 4 );
REQUIRE( pack.dsize() == 2 );
for( int i = 0; i < pack.asize(); i++ )
for( size_t i = 0; i < pack.asize(); i++ )
{
REQUIRE( pack.a_dat[i].val == i + 1 );
}
......
extensions/UNetUDP/tests/tests_individual_process.cc
View file @ 29709852
......@@ -15,6 +15,8 @@ using namespace UniSetExtensions;
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -31,8 +33,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
uniset_init(argc, argv);
UNetExchange* unet = UNetExchange::init_unetexchange(argc, argv, getSharedMemoryID());
......
extensions/UNetUDP/tests/tests_with_sm.cc
View file @ 29709852
......@@ -15,6 +15,8 @@ using namespace UniSetExtensions;
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -31,8 +33,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
auto conf = uniset_init(argc, argv);
bool apart = findArgParam("--apart", argc, argv) != -1;
......
extensions/UNetUDP/unet-udp-tester.cc
View file @ 29709852
......@@ -166,9 +166,6 @@ int main(int argc, char* argv[])
return -1;
}
if( tout < 0 )
tout = TIMEOUT_INF;
ost::Thread::setException(ost::Thread::throwException);
try
......
extensions/include/IOBase.h
View file @ 29709852
......@@ -65,6 +65,12 @@ struct IOBase
si.node = UniSetTypes::DefaultObjectId;
cal.minRaw = cal.maxRaw = cal.minCal = cal.maxCal = cal.precision = 0;
ti.invert = false;
ti.hilimit = 0;
ti.lowlimit = 0;
ti.id = UniSetTypes::DefaultObjectId;
ti.state = IONotifyController_i::NormalThreshold;
ti.tv_sec = 0;
ti.tv_usec = 0;
}
bool check_channel_break( long val ); /*!< проверка обрыва провода */
......
extensions/include/MTR.h
View file @ 29709852
......@@ -293,6 +293,7 @@ namespace MTR
T5( long v )
{
raw.lval = v;
val = raw.u2.val * pow( (long)10, (long)raw.u2.exp );
}
T5( const ModbusRTU::ModbusData* data, int size )
......@@ -355,6 +356,7 @@ namespace MTR
T6( long v )
{
raw.lval = v;
val = raw.u2.val * pow( (long)10, (long)raw.u2.exp );
}
T6( const ModbusRTU::ModbusData* data, int size )
......@@ -381,7 +383,7 @@ namespace MTR
return mtT6;
}
// ------------------------------------------
double val;
double val = { 0.0 };
T6mem raw;
};
std::ostream& operator<<(std::ostream& os, T6& t );
......@@ -417,6 +419,7 @@ namespace MTR
T7( const long v )
{
raw.lval = v;
val = raw.u2.val * pow( (long)10, (long) - 4 );
}
T7( const ModbusRTU::ModbusData* data, int size )
......@@ -443,7 +446,7 @@ namespace MTR
return mtT7;
}
// ------------------------------------------
double val;
double val = { 0.0 };
T7mem raw;
};
std::ostream& operator<<(std::ostream& os, T7& t );
......
extensions/lib/SingleProcess.cc
View file @ 29709852
......@@ -69,6 +69,8 @@ void SingleProcess::set_signals( bool ask )
struct sigaction act, oact;
sigemptyset(&act.sa_mask);
act.sa_flags = SA_RESETHAND;
// добавляем сигналы, которые будут игнорироваться
// при обработке сигнала
sigaddset(&act.sa_mask, SIGINT);
......
extensions/tests/tests_with_conf.cc
View file @ 29709852
......@@ -6,10 +6,10 @@
int main( int argc, char* argv[] )
{
Catch::Session session;
try
{
Catch::Session session;
UniSetTypes::uniset_init(argc, argv);
int returnCode = session.applyCommandLine( argc, argv, Catch::Session::OnUnusedOptions::Ignore );
......
extensions/tests/tests_with_sm.cc
View file @ 29709852
......@@ -45,6 +45,8 @@ shared_ptr<SMInterface> smiInstance()
// --------------------------------------------------------------------------
int main(int argc, char* argv[] )
{
try
{
Catch::Session session;
if( argc > 1 && ( strcmp(argv[1], "--help") == 0 || strcmp(argv[1], "-h") == 0 ) )
......@@ -61,8 +63,6 @@ int main(int argc, char* argv[] )
if( returnCode != 0 ) // Indicates a command line error
return returnCode;
try
{
auto conf = uniset_init(argc, argv);
shm = SharedMemory::init_smemory(argc, argv);
......
include/HourGlass.h
View file @ 29709852
......@@ -102,10 +102,7 @@ class HourGlass
if( cur > _size )
cur = _size;
_sand -= cur;
if( _sand < 0 )
_sand = 0;
_sand = ( _sand > cur ) ? (_sand - cur) : 0;
t.setTiming(cur);
}
......
include/modbus/ModbusTypes.h
View file @ 29709852
......@@ -542,7 +542,7 @@ namespace ModbusRTU
// преобразовании в ModbusMessage.
// Делать что-типа memcpy(buf,this,sizeof(*this)); будет не верно.
// Используйте специальную функцию transport_msg()
int count = { 0 }; /*!< фактическое количество данных в сообщении */
size_t count = { 0 }; /*!< фактическое количество данных в сообщении */
};
std::ostream& operator<<(std::ostream& os, ReadOutputRetMessage& m );
......@@ -633,7 +633,7 @@ namespace ModbusRTU
// преобразовании в ModbusMessage.
// Делать что-типа memcpy(buf,this,sizeof(*this)); будет не верно.
// Используйте специальную функцию transport_msg()
int count = { 0 }; /*!< фактическое количество данных в сообщении */
size_t count = { 0 }; /*!< фактическое количество данных в сообщении */
};
std::ostream& operator<<(std::ostream& os, ReadInputRetMessage& m );
......@@ -1291,7 +1291,7 @@ namespace ModbusRTU
// преобразовании в ModbusMessage.
// Делать что-типа memcpy(buf,this,sizeof(*this)); будет не верно.
// Используйте специальную функцию transport_msg()
int count = { 0 }; /*!< фактическое количество данных в сообщении */
size_t count = { 0 }; /*!< фактическое количество данных в сообщении */
};
std::ostream& operator<<(std::ostream& os, JournalCommandRetMessage& m );
......@@ -1440,7 +1440,7 @@ namespace ModbusRTU
// Это поле не входит в стандарт modbus
// оно вспомогательное и игнорируется при
// преобразовании в ModbusMessage.
unsigned int count = { 0 }; /*!< фактическое количество данных в сообщении */
size_t count = { 0 }; /*!< фактическое количество данных в сообщении */
};
// -----------------------------------------------------------------------
......@@ -1484,7 +1484,7 @@ namespace ModbusRTU
bool checkFormat();
// это поле служебное и не используется в релальном обмене
int count = { 0 }; /*!< фактическое количество данных */
size_t count = { 0 }; /*!< фактическое количество данных */
};
std::ostream& operator<<(std::ostream& os, ReadFileRecordMessage& m );
......
python/lib/pyUniSet/PyUInterface.cc
View file @ 29709852
......@@ -28,9 +28,9 @@ void pyUInterface::uniset_init( int argc, char* argv[], const char* xmlfile )thr
{
throw UException(ex.what());
}
catch( ... )
catch( std::exception& ex )
{
throw UException();
throw UException(ex.what());
}
}
//---------------------------------------------------------------------------
......@@ -70,9 +70,9 @@ long pyUInterface::getValue( long id )throw(UException)
{
throw UException(ex.what());
}
catch( ... )
catch( std::exception& ex )
{
throw UException("(getValue): catch...");
throw UException(ex.what());
}
throw UException("(getValue): unknown error");
......@@ -114,9 +114,9 @@ void pyUInterface::setValue( long id, long val )throw(UException)
{
throw UException(ex.what());
}
catch( ... )
catch( std::exception& ex )
{
throw UException("(setValue): catch...");
throw UException(ex.what());
}
}
//---------------------------------------------------------------------------
......@@ -135,7 +135,7 @@ const char* pyUInterface::getName( long id )
auto conf = UniSetTypes::uniset_conf();
if( conf )
return conf->oind->getMapName(id).c_str();
return UniSetTypes::uni_strdup(conf->oind->getMapName(id));
return "";
}
......@@ -145,7 +145,7 @@ const char* pyUInterface::getShortName( long id )
auto conf = UniSetTypes::uniset_conf();
if( conf )
return ORepHelpers::getShortName(conf->oind->getMapName(id)).c_str();
return UniSetTypes::uni_strdup(ORepHelpers::getShortName(conf->oind->getMapName(id)));
return "";
}
......@@ -155,7 +155,7 @@ const char* pyUInterface::getTextName( long id )
auto conf = UniSetTypes::uniset_conf();
if( conf )
return conf->oind->getTextName(id).c_str();
return UniSetTypes::uni_strdup(conf->oind->getTextName(id));
return "";
}
......@@ -165,7 +165,7 @@ const char* pyUInterface::getConfFileName()
auto conf = UniSetTypes::uniset_conf();
if( conf )
return conf->getConfFileName().c_str();
return UniSetTypes::uni_strdup(conf->getConfFileName());
return "";
......
python/lib/pyUniSet/UConnector.cc
View file @ 29709852
......@@ -51,7 +51,7 @@ const char* UConnector::getConfFileName()
{
// return xmlfile;
if( conf )
return conf->getConfFileName().c_str();
return UniSetTypes::uni_strdup(conf->getConfFileName());
return "";
......@@ -73,12 +73,10 @@ long UConnector::getValue( long id, long node )throw(UException)
{
throw UException(ex.what());
}
catch( ... )
catch( std::exception& ex )
{
throw UException("(getValue): catch...");
throw UException(ex.what());
}
throw UException("(getValue): unknown error");
}
//---------------------------------------------------------------------------
void UConnector::setValue( long id, long val, long node )throw(UException)
......@@ -98,9 +96,9 @@ void UConnector::setValue( long id, long val, long node )throw(UException)
{
throw UException(ex.what());
}
catch( ... )
catch( std::exception& ex )
{
throw UException("(setValue): catch...");
throw UException(ex.what());
}
}
//---------------------------------------------------------------------------
......@@ -123,7 +121,7 @@ long UConnector::getNodeID( const char* name )
const char* UConnector::getName( long id )
{
if( conf )
return conf->oind->getMapName(id).c_str();
return UniSetTypes::uni_strdup(conf->oind->getMapName(id));
return "";
}
......@@ -131,7 +129,7 @@ const char* UConnector::getName( long id )
const char* UConnector::getShortName( long id )
{
if( conf )
return ORepHelpers::getShortName(conf->oind->getMapName(id)).c_str();
return UniSetTypes::uni_strdup(ORepHelpers::getShortName(conf->oind->getMapName(id)));
return "";
}
......@@ -139,7 +137,7 @@ const char* UConnector::getShortName( long id )
const char* UConnector::getTextName( long id )
{
if( conf )
return conf->oind->getTextName(id).c_str();
return UniSetTypes::uni_strdup(conf->oind->getTextName(id));
return "";
}
......
src/Communications/Modbus/ModbusClient.cc
View file @ 29709852
......@@ -997,8 +997,8 @@ mbErrCode ModbusClient::recv_pdu( ModbusByte qfunc, ModbusMessage& rbuf, timeout
while( (rlen+2) < sizeof(rbuf) && onum < mPreRDI.objNum )
{
// сперва получаем два байта, для определения длины последующих данных
int szDataLen = 2; // object id + len
int rlen1 = getNextData((unsigned char*)(&(rbuf.data[rlen])), szDataLen);
size_t szDataLen = 2; // object id + len
size_t rlen1 = getNextData((unsigned char*)(&(rbuf.data[rlen])), szDataLen);
if( rlen1 < szDataLen )
{
......
src/Communications/Modbus/ModbusTypes.cc
View file @ 29709852
......@@ -260,7 +260,7 @@ ModbusMessage ErrorRetMessage::transport_msg()
memcpy(&mm.data, &ecode, sizeof(ecode));
int ind = sizeof(ecode);
size_t ind = sizeof(ecode);
// пересчитываем CRC по перевёрнутым данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + ind );
......@@ -308,7 +308,7 @@ ModbusMessage ReadCoilMessage::transport_msg()
// копируем данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(count) };
int last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
size_t last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
// копируем
memcpy(mm.data, &d, last);
......@@ -509,7 +509,7 @@ void ReadCoilRetMessage::init( ModbusMessage& m )
// -------------------------------------------------------------------------
int ReadCoilRetMessage::getDataLen( ModbusMessage& m )
{
if( m.len < 0 )
if( m.len == 0 )
return 0;
return m.data[0];
......@@ -572,7 +572,7 @@ ModbusMessage ReadCoilRetMessage::transport_msg()
memcpy(&mm, this, szModbusHeader);
memcpy(&mm.data, &bcnt, sizeof(bcnt));
int ind = sizeof(bcnt);
size_t ind = sizeof(bcnt);
// копируем данные
memcpy(&(mm.data[ind]), data, bcnt);
......@@ -627,7 +627,7 @@ ModbusMessage ReadInputStatusMessage::transport_msg()
// копируем данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(count) };
int last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
size_t last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
// копируем
memcpy(mm.data, &d, last);
......@@ -711,7 +711,7 @@ void ReadInputStatusRetMessage::init( ModbusMessage& m )
// -------------------------------------------------------------------------
int ReadInputStatusRetMessage::getDataLen( ModbusMessage& m )
{
if( m.len < 0 )
if( m.len == 0 )
return 0;
return m.data[0];
......@@ -774,7 +774,7 @@ ModbusMessage ReadInputStatusRetMessage::transport_msg()
memcpy(&mm, this, szModbusHeader);
memcpy(&mm.data, &bcnt, sizeof(bcnt));
int ind = sizeof(bcnt);
size_t ind = sizeof(bcnt);
// копируем данные
memcpy(&(mm.data[ind]), data, bcnt);
......@@ -830,7 +830,7 @@ ModbusMessage ReadOutputMessage::transport_msg()
// копируем данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(count) };
int last(sizeof(d)); // индекс в массиве данных ( байтовый массив!!! )
size_t last(sizeof(d)); // индекс в массиве данных ( байтовый массив!!! )
// копируем
memcpy(mm.data, &d, last);
......@@ -929,7 +929,7 @@ void ReadOutputRetMessage::init( ModbusMessage& m )
// -------------------------------------------------------------------------
int ReadOutputRetMessage::getDataLen( ModbusMessage& m )
{
if( m.len < 0 )
if( m.len == 0 )
return 0;
return m.data[0];
......@@ -974,7 +974,7 @@ ModbusMessage ReadOutputRetMessage::transport_msg()
// копируем заголовок и данные
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
bcnt = count * sizeof(ModbusData);
// copy bcnt
......@@ -982,14 +982,12 @@ ModbusMessage ReadOutputRetMessage::transport_msg()
ind += sizeof(bcnt);
// копируем данные
// int dlen = count*sizeof(ModbusData);
// прямое копирование
// memcpy(&(mm.data[sizeof(bcnt)]),data,dlen);
if( count > 0 )
{
// Создаём временно массив, переворачиваем байты
ModbusData* dtmp = new ModbusData[count];
for( ssize_t i = 0; i < count; i++ )
for( size_t i = 0; i < count; i++ )
dtmp[i] = SWAPSHORT(data[i]);
// копируем
......@@ -998,12 +996,10 @@ ModbusMessage ReadOutputRetMessage::transport_msg()
delete[] dtmp;
ind += bcnt;
}
// пересчитываем CRC по перевёрнутым данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + sizeof(bcnt) + bcnt );
// crc = SWAPSHORT(crc);
// копируем CRC (последний элемент). Без переворачивания...
memcpy(&(mm.data[ind]), &crc, szCRC);
ind += szCRC;
......@@ -1051,7 +1047,7 @@ ModbusMessage ReadInputMessage::transport_msg()
// копируем данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(count) };
int last(sizeof(d)); // индекс в массиве данных ( байтовый массив!!! )
size_t last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
// копируем
memcpy(mm.data, &d, last);
......@@ -1145,13 +1141,13 @@ void ReadInputRetMessage::init( ModbusMessage& m )
void ReadInputRetMessage::swapData()
{
// переворачиваем данные
for( ssize_t i = 0; i < count; i++ )
for( size_t i = 0; i < count; i++ )
data[i] = SWAPSHORT(data[i]);
}
// -------------------------------------------------------------------------
int ReadInputRetMessage::getDataLen( ModbusMessage& m )
{
if( m.len < 0 )
if( m.len == 0 )
return 0;
return m.data[0];
......@@ -1191,17 +1187,19 @@ ModbusMessage ReadInputRetMessage::transport_msg()
// копируем заголовок и данные
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
bcnt = count * sizeof(ModbusData);
// copy bcnt
memcpy(&mm.data, &bcnt, sizeof(bcnt));
ind += sizeof(bcnt);
if( count > 0 )
{
// Создаём временно массив, переворачиваем байты
ModbusData* dtmp = new ModbusData[count];
for( ssize_t i = 0; i < count; i++ )
for( size_t i = 0; i < count; i++ )
dtmp[i] = SWAPSHORT(data[i]);
// копируем
......@@ -1210,7 +1208,7 @@ ModbusMessage ReadInputRetMessage::transport_msg()
delete[] dtmp;
ind += bcnt;
}
// пересчитываем CRC по перевёрнутым данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + sizeof(bcnt) + bcnt );
......@@ -1315,7 +1313,7 @@ ModbusMessage ForceCoilsMessage::transport_msg()
// копируем заголовок
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
// данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(quant) };
......@@ -1403,7 +1401,7 @@ size_t ForceCoilsMessage::szData()
// -------------------------------------------------------------------------
int ForceCoilsMessage::getDataLen( ModbusMessage& m )
{
if( m.len < 0 )
if( m.len == 0 )
return 0;
ForceCoilsMessage wm(m);
......@@ -1490,7 +1488,7 @@ ModbusMessage ForceCoilsRetMessage::transport_msg()
// данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(quant) };
int last(sizeof(d));
size_t last = sizeof(d);
// копируем
memcpy(mm.data, &d, last);
......@@ -1550,7 +1548,7 @@ ModbusMessage WriteOutputMessage::transport_msg()
// копируем заголовок
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
// данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(quant) };
......@@ -1567,7 +1565,7 @@ ModbusMessage WriteOutputMessage::transport_msg()
// Создаём временно массив, переворачиваем байты
ModbusData* dtmp = new ModbusData[quant];
for( ssize_t i = 0; i < quant; i++ )
for( size_t i = 0; i < quant; i++ )
dtmp[i] = SWAPSHORT(data[i]);
// копируем данные
......@@ -1749,7 +1747,7 @@ ModbusMessage WriteOutputRetMessage::transport_msg()
// данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(quant) };
int last(sizeof(d));
size_t last = sizeof(d);
// копируем
memcpy(mm.data, &d, last);
......@@ -1788,7 +1786,7 @@ ModbusMessage ForceSingleCoilMessage::transport_msg()
ModbusMessage mm;
memcpy(&mm, this, szModbusHeader);
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(data) };
int last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
size_t last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
memcpy(mm.data, &d, last);
// пересчитываем CRC по перевёрнутым данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + last );
......@@ -1917,7 +1915,7 @@ ModbusMessage ForceSingleCoilRetMessage::transport_msg()
// копируем данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(data) };
int last(sizeof(d)); // индекс в массиве данных ( байтовый массив!!! )
size_t last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
// копируем
memcpy(mm.data, &d, last);
......@@ -1959,7 +1957,7 @@ ModbusMessage WriteSingleOutputMessage::transport_msg()
ModbusMessage mm;
memcpy(&mm, this, szModbusHeader);
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(data) };
int last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
size_t last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
memcpy(mm.data, &d, last);
// пересчитываем CRC по перевёрнутым данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + last );
......@@ -2095,7 +2093,7 @@ ModbusMessage WriteSingleOutputRetMessage::transport_msg()
// копируем данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(start), SWAPSHORT(data) };
int last(sizeof(d)); // индекс в массиве данных ( байтовый массив!!! )
size_t last = sizeof(d); // индекс в массиве данных ( байтовый массив!!! )
// копируем
memcpy(mm.data, &d, last);
......@@ -2217,7 +2215,7 @@ void DiagnosticMessage::init( ModbusMessage& m )
// -------------------------------------------------------------------------
int DiagnosticMessage::getDataLen( ModbusMessage& m )
{
if( m.len < 0 )
if( m.len == 0 )
return 0;
// data[0] = subfunction
......@@ -2267,12 +2265,14 @@ ModbusMessage DiagnosticMessage::transport_msg()
// копируем заголовок и данные
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
// copy bcnt
ModbusData d = SWAPSHORT(subf);
memcpy(&mm.data, &d, sizeof(d));
ind += sizeof(subf);
if( count > 0 )
{
// Создаём временно массив, переворачиваем байты
ModbusData* dtmp = new ModbusData[count];
......@@ -2285,6 +2285,7 @@ ModbusMessage DiagnosticMessage::transport_msg()
delete[] dtmp;
ind += sizeof(ModbusData) * count;
}
// пересчитываем CRC по перевёрнутым данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + sizeof(subf) + sizeof(ModbusData) * count );
......@@ -2380,7 +2381,7 @@ ModbusMessage MEIMessageRDI::transport_msg()
mm.data[0] = type;
mm.data[1] = devID;
mm.data[2] = objID;
int ind = 3;
size_t ind = 3;
// пересчитываем CRC по перевёрнутым данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + ind );
......@@ -2606,7 +2607,7 @@ ModbusMessage MEIMessageRetRDI::transport_msg()
mm.data[3] = mf;
mm.data[4] = objID;
mm.data[5] = objNum;
int ind = 6;
size_t ind = 6;
for( auto it = dlist.begin(); it != dlist.end() && ind <= MAXLENPACKET; ++it )
{
......@@ -2762,7 +2763,7 @@ ModbusMessage JournalCommandRetMessage::transport_msg()
// копируем заголовок и данные
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
bcnt = count * sizeof(ModbusData);
// copy bcnt
......@@ -2772,10 +2773,12 @@ ModbusMessage JournalCommandRetMessage::transport_msg()
// --------------------
// копируем данные
// --------------------
if( count > 0 )
{
// копирование с переворотом данных (для ModbusData)
ModbusData* dtmp = new ModbusData[count];
for( ssize_t i = 0; i < count; i++ )
for( size_t i = 0; i < count; i++ )
dtmp[i] = SWAPSHORT(data[i]);
// копируем
......@@ -2784,6 +2787,7 @@ ModbusMessage JournalCommandRetMessage::transport_msg()
delete[] dtmp;
ind += bcnt;
}
// пересчитываем CRC по данным
ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader + ind );
......@@ -3025,7 +3029,7 @@ ModbusMessage SetDateTimeMessage::transport_msg()
mm.data[5] = year;
mm.data[6] = century;
*/
int bcnt = 7;
size_t bcnt = 7;
memcpy( mm.data, &hour, bcnt );
// пересчитываем CRC
......@@ -3099,7 +3103,7 @@ ModbusMessage SetDateTimeRetMessage::transport_msg()
mm.data[5] = year;
mm.data[6] = century;
*/
int bcnt = 7;
size_t bcnt = 7;
memcpy( mm.data, &hour, bcnt );
// пересчитываем CRC
......@@ -3211,7 +3215,7 @@ ModbusMessage RemoteServiceRetMessage::transport_msg()
// копируем заголовок и данные
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
bcnt = count * sizeof(ModbusByte);
// copy bcnt
......@@ -3277,7 +3281,7 @@ void ReadFileRecordMessage::init( ModbusMessage& m )
count = bcnt / sizeof(SubRequest);
for( ssize_t i = 0; i < count; i++ )
for( size_t i = 0; i < count; i++ )
{
data[i].numfile = SWAPSHORT(data[i].numfile);
data[i].numrec = SWAPSHORT(data[i].numrec);
......@@ -3329,7 +3333,7 @@ ModbusMessage FileTransferMessage::transport_msg()
// копируем данные (переворачиваем байты)
ModbusData d[2] = { SWAPSHORT(numfile), SWAPSHORT(numpacket) };
int last = sizeof(d);
size_t last = sizeof(d);
memcpy(mm.data, &d, last);
// пересчитываем CRC по перевёрнутым данным
......@@ -3469,7 +3473,7 @@ ModbusMessage FileTransferRetMessage::transport_msg()
// копируем заголовок и данные
memcpy(&mm, this, szModbusHeader);
int ind = 0;
size_t ind = 0;
bcnt = szData() - szCRC - 1; // -1 - это сам байт содержащий количество байт (bcnt)...
// copy bcnt
......
src/Various/Configuration.cc
View file @ 29709852
......@@ -928,7 +928,6 @@ namespace UniSetTypes
// -------------------------------------------------------------------------
string Configuration::getRepSectionName( const string& sec, xmlNode* secnode )
{
secnode = 0;
xmlNode* node = unixml->findNode(unixml->getFirstNode(), sec);
if( node == NULL )
......
tests/tests_with_conf.cc
View file @ 29709852
......@@ -6,10 +6,10 @@
int main( int argc, char* argv[] )
{
Catch::Session session;
try
{
Catch::Session session;
UniSetTypes::uniset_init(argc, argv);
int returnCode = session.applyCommandLine( argc, argv, Catch::Session::OnUnusedOptions::Ignore );
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment