[LyX/master] Check for NULL pointer earlier

[Juergen Spitzmueller]

commit 5ec247a957f446ed0bb26fe958d11aec95c3325c
Author: Juergen Spitzmueller <spitz at lyx.org>
Date:   Sun Oct 20 08:05:06 2024 +0200

    Check for NULL pointer earlier
    
    Reported by Coverity
---
 src/frontends/qt/GuiBibtex.cpp | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/frontends/qt/GuiBibtex.cpp b/src/frontends/qt/GuiBibtex.cpp
index eeb8910cdd..a3b624a84f 100644
--- a/src/frontends/qt/GuiBibtex.cpp
+++ b/src/frontends/qt/GuiBibtex.cpp
@@ -186,6 +186,8 @@ void GuiBibtex::selUpdated()
 		// check for current file encodings
 		for (int i = 0; i != selected_model_.rowCount(); ++i) {
 			QStandardItem const * key = selected_model_.item(i, 0);
+			if (!key)
+				continue;
 			QComboBox * cb = qobject_cast<QComboBox*>(selectedLV->indexWidget(selected_model_.index(i, 1)));
 			QString fenc = cb ? cb->itemData(cb->currentIndex()).toString() : QString();
 			if (fenc.isEmpty())
@@ -193,7 +195,7 @@ void GuiBibtex::selUpdated()
 			else
 				cached_file_encodings_[key->text()] = fenc;
 			docstring const enc = qstring_to_ucs4(key->text()) + " " + qstring_to_ucs4(fenc);
-			if (key && !key->text().isEmpty() && !fenc.isEmpty() && fenc != "general")
+			if (!key->text().isEmpty() && !fenc.isEmpty() && fenc != "general")
 				nfe.push_back(enc);
 		}
 	}
@@ -326,10 +328,12 @@ void GuiBibtex::updateFileEncodings()
 	vector<docstring> nfe;
 	for (int i = 0; i != selected_model_.rowCount(); ++i) {
 		QStandardItem const * key = selected_model_.item(i, 0);
+		if (!key)
+			continue;
 		QComboBox * cb = qobject_cast<QComboBox*>(selectedLV->indexWidget(selected_model_.index(i, 1)));
 		QString fenc = cb ? cb->itemData(cb->currentIndex()).toString() : QString();
 		docstring const enc = qstring_to_ucs4(key->text()) + " " + qstring_to_ucs4(fenc);
-		if (key && !key->text().isEmpty() && !fenc.isEmpty() && fenc != "general")
+		if (!key->text().isEmpty() && !fenc.isEmpty() && fenc != "general")
 			nfe.push_back(enc);
 	}
 	if (!nfe.empty()) {