Fix crash when non-exist sheetName passed to renameSheet()

This commit is contained in:
Debao Zhang
2014-11-07 12:45:42 +08:00
parent 1a44a04171
commit 9a205f1698
+4
View File
@@ -290,6 +290,10 @@ bool Workbook::setActiveSheet(int index)
bool Workbook::renameSheet(int index, const QString &name) bool Workbook::renameSheet(int index, const QString &name)
{ {
Q_D(Workbook); Q_D(Workbook);
if (index < 0 || index >= d->sheets.size())
return false;
//If user given an already in-used name, return false //If user given an already in-used name, return false
for (int i=0; i<d->sheets.size(); ++i) { for (int i=0; i<d->sheets.size(); ++i) {
if (d->sheets[i]->sheetName() == name) if (d->sheets[i]->sheetName() == name)