Severe bugs in ConfiguratorWindow::getEditedConfiguration(), where the bitmap fields are not read properly

[samuelfmlourenco]

Inside configuratorwindow.cpp, lines 601 to 604, in getEditedConfiguration(), the values should be treated as base-16. Unfortunately, this went unnoticed. Currently we have:

void ConfiguratorWindow::getEditedConfiguration()
{
    ...
    editedConfig_.pinconfig.sspndlvl = static_cast<quint16>(ui->lineEditSuspendLevel->text().toUInt());
    editedConfig_.pinconfig.sspndmode = static_cast<quint16>(ui->lineEditSuspendMode->text().toUInt());
    editedConfig_.pinconfig.wkupmask = static_cast<quint16>(ui->lineEditResumeMask->text().toUInt());
    editedConfig_.pinconfig.wkupmatch = static_cast<quint16>(ui->lineEditResumeMatch->text().toUInt());
    ...
}

The values read from the line edit boxes are represented in hexadecimal, but they are being processed as if they were decimal. This obviously leads to severe conversion errors. The fix is simple:

void ConfiguratorWindow::getEditedConfiguration()
{
    ...
    editedConfig_.pinconfig.sspndlvl = static_cast<quint16>(ui->lineEditSuspendLevel->text().toUInt(nullptr, 16));
    editedConfig_.pinconfig.sspndmode = static_cast<quint16>(ui->lineEditSuspendMode->text().toUInt(nullptr, 16));
    editedConfig_.pinconfig.wkupmask = static_cast<quint16>(ui->lineEditResumeMask->text().toUInt(nullptr, 16));
    editedConfig_.pinconfig.wkupmatch = static_cast<quint16>(ui->lineEditResumeMatch->text().toUInt(nullptr, 16));
    ...
}

[samuelfmlourenco]

Fixed in version 3.0.