Ticket #6591: mythtv-6591-check_return_value_of_MSqlQuery_exec-important.patch

File mythtv-6591-check_return_value_of_MSqlQuery_exec-important.patch, 4.1 KB (added by sphery <mtdean@…>, 12 years ago)

Most important fixes in mythtv--cases that rely on QSqlQuery::lastError() and ignore MSqlQuery::exec()

  • libs/libmythtv/dbcheck.cpp

    old new  
    359359
    360360    QString thequery = "DELETE FROM settings WHERE value='DBSchemaVer';";
    361361    query.prepare(thequery);
    362     query.exec();
    363362
    364     if (query.lastError().type() != QSqlError::NoError)
     363    if (!query.exec())
    365364    {
    366365        QString msg =
    367366            QString("DB Error (Deleting old DB version number): \n"
    … …  
    377376    thequery = QString("INSERT INTO settings (value, data, hostname) "
    378377                       "VALUES ('DBSchemaVer', %1, NULL);").arg(newnumber);
    379378    query.prepare(thequery);
    380     query.exec();
    381379
    382     if (query.lastError().type() != QSqlError::NoError)
     380    if (!query.exec())
    383381    {
    384382        QString msg =
    385383            QString("DB Error (Setting new DB version number): \n"

  • libs/libmythtv/scheduledrecording.cpp

    old new  
    991991    thequery ="SELECT GET_LOCK(:LOCK, 2);";
    992992    query.prepare(thequery);
    993993    query.bindValue(":LOCK", "DiffSchedule");
    994     query.exec();
    995     if (query.lastError().type() != QSqlError::NoError)
     994    if (!query.exec())
    996995    {
    997996        QString msg =
    998997            QString("DB Error (Obtaining lock in testRecording): \n"
    … …  
    10061005
    10071006    thequery = QString("DROP TABLE IF EXISTS %1;").arg(ttable);
    10081007    query.prepare(thequery);
    1009     query.exec();
    1010     if (query.lastError().type() != QSqlError::NoError)
     1008    if (!query.exec())
    10111009    {
    10121010        QString msg =
    10131011            QString("DB Error (deleting old table in testRecording): \n"
    … …  
    10201018
    10211019    thequery = QString("CREATE TABLE %1 SELECT * FROM record;").arg(ttable);
    10221020    query.prepare(thequery);
    1023     query.exec();
    1024     if (query.lastError().type() != QSqlError::NoError)
     1021    if (!query.exec())
    10251022    {
    10261023        QString msg =
    10271024            QString("DB Error (create new table): \n"
    … …  
    10521049    thequery = "SELECT RELEASE_LOCK(:LOCK);";
    10531050    query.prepare(thequery);
    10541051    query.bindValue(":LOCK", "DiffSchedule");
    1055     query.exec();
    1056     if (query.lastError().type() != QSqlError::NoError)
     1052    if (!query.exec())
    10571053    {
    10581054        QString msg =
    10591055            QString("DB Error (free lock): \n"

  • programs/mythfrontend/custompriority.cpp

    old new  
    388388    thequery = "SELECT GET_LOCK(:LOCK, 2);";
    389389    query.prepare(thequery);
    390390    query.bindValue(":LOCK", "DiffSchedule");
    391     query.exec();
    392     if (query.lastError().type() != QSqlError::NoError)
     391    if (!query.exec())
    393392    {
    394393        QString msg =
    395394            QString("DB Error (Obtaining lock in testRecording): \n"
    … …  
    402401
    403402    thequery = QString("DROP TABLE IF EXISTS %1;").arg(ttable);
    404403    query.prepare(thequery);
    405     query.exec();
    406     if (query.lastError().type() != QSqlError::NoError)
     404    if (!query.exec())
    407405    {
    408406        QString msg =
    409407            QString("DB Error (deleting old table in testRecording): \n"
    … …  
    417415    thequery = QString("CREATE TABLE %1 SELECT * FROM powerpriority;")
    418416                       .arg(ttable);
    419417    query.prepare(thequery);
    420     query.exec();
    421     if (query.lastError().type() != QSqlError::NoError)
     418    if (!query.exec())
    422419    {
    423420        QString msg =
    424421            QString("DB Error (create new table): \n"
    … …  
    457454    thequery = "SELECT RELEASE_LOCK(:LOCK);";
    458455    query.prepare(thequery);
    459456    query.bindValue(":LOCK", "DiffSchedule");
    460     query.exec();
    461     if (query.lastError().type() != QSqlError::NoError)
     457    if (!query.exec())
    462458    {
    463459        QString msg =
    464460            QString("DB Error (free lock): \n"