[Scummvm-git-logs] scummvm master -> 99342fc1e38199cddfd11e4e9da784eabea29312

dreammaster noreply at scummvm.org
Tue Sep 30 10:25:57 UTC 2025


This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://api.github.com/repos/scummvm/scummvm .

Summary:
99342fc1e3 BAGEL: MINIGAMES: Fix ArtParts Coverity warnings


Commit: 99342fc1e38199cddfd11e4e9da784eabea29312
    https://github.com/scummvm/scummvm/commit/99342fc1e38199cddfd11e4e9da784eabea29312
Author: Paul Gilbert (dreammaster at scummvm.org)
Date: 2025-09-30T03:25:51-07:00

Commit Message:
BAGEL: MINIGAMES: Fix ArtParts Coverity warnings

Changed paths:
    engines/bagel/hodjnpodj/artparts/artparts.cpp
    engines/bagel/hodjnpodj/artparts/optndlg.h


diff --git a/engines/bagel/hodjnpodj/artparts/artparts.cpp b/engines/bagel/hodjnpodj/artparts/artparts.cpp
index 2dde605581c..53db221363d 100644
--- a/engines/bagel/hodjnpodj/artparts/artparts.cpp
+++ b/engines/bagel/hodjnpodj/artparts/artparts.cpp
@@ -903,7 +903,7 @@ void CMainWindow::OnRButtonDown(unsigned int nFlags, CPoint point) {
 			}
 
 			else if (bSwitched) {                        // Switch back the two areas
-				SwitchAreas(DstRect, SrcRect);
+				SwitchAreas(SrcRect, DstRect);
 				bSwitched = false;                          // Don't allow second Undo
 			}
 
@@ -1313,6 +1313,8 @@ bool CMainWindow::LoadArtWork() {
 	}
 	inFile.getline(chNumEntries, sizeof(chNumEntries));     // read number of names in file
 	nNumEntries = atoi(chNumEntries);
+	assert(nNumEntries > 0);
+
 	pick = nLastPick;
 	while (pick == nLastPick) {
 		pick = (brand() % nNumEntries) + 1;
@@ -1741,17 +1743,12 @@ void CMainWindow::MyFocusRect(CDC *pDC, CRect rect, int nDrawMode) {
 	pDC->SetROP2(OldDrawMode);                   // Set pen mode back to old state
 	(*pDC).SelectPalette(pPalOld, false);           // Select back the old palette
 
-	if (pMyBrush != nullptr) {                         // If the brush was constructed, delete it
-		pMyBrush->DeleteObject();
-		delete pMyBrush;
-	}
+	pMyBrush->DeleteObject();
+	delete pMyBrush;
 
-	if (pMyPen != nullptr) {                           // If the pen was constructed, delete it
-		pMyPen->DeleteObject();
-		delete pMyPen;
-	}
-
-} // End MyFocusRect()
+	pMyPen->DeleteObject();
+	delete pMyPen;
+}
 
 /*****************************************************************
  *
diff --git a/engines/bagel/hodjnpodj/artparts/optndlg.h b/engines/bagel/hodjnpodj/artparts/optndlg.h
index 41366e904cb..243ac33b5cc 100644
--- a/engines/bagel/hodjnpodj/artparts/optndlg.h
+++ b/engines/bagel/hodjnpodj/artparts/optndlg.h
@@ -43,17 +43,17 @@ public:
 	void UpdateScrollbars();
 	void ClearDialogImage();
 
-	int m_nTime;
-	int m_nRows;
-	int m_nColumns;
-	int m_nMins;
-	int m_nSecs;
-	float   m_nScore;
-	bool    m_bFramed;
-	int m_nNumParts;
+	int m_nTime = 0;
+	int m_nRows = 0;
+	int m_nColumns = 0;
+	int m_nMins = 0;
+	int m_nSecs = 0;
+	float   m_nScore = 0.0;
+	bool    m_bFramed = false;
+	int m_nNumParts = 0;
 
-	int nCFacs;
-	int nRFacs;
+	int nCFacs = 0;
+	int nRFacs = 0;
 
 // Dialog Data
 	//{{AFX_DATA(COptnDlg)




More information about the Scummvm-git-logs mailing list