reject empty field names in QgsVectorLayer::addAttribute

jef-n · alexbruy · commit 78d2ee71b192 · 2012-02-11T17:22:48.000+02:00
diff --git a/src/core/qgsvectorlayer.cpp b/src/core/qgsvectorlayer.cpp
@@ -3324,6 +3324,9 @@ bool QgsVectorLayer::addAttribute( const QgsField &field )
   if ( !isEditable() )
     return false;
 
+  if ( field.name().isEmpty() )
+    return false;
+
   for ( QgsFieldMap::const_iterator it = mUpdatedFields.begin(); it != mUpdatedFields.end(); it++ )
   {
     if ( it.value().name() == field.name() )

Original file line number	Diff line number	Diff line change
`@@ -3324,6 +3324,9 @@ bool QgsVectorLayer::addAttribute( const QgsField &field )`
`3324`	`3324`	`if ( !isEditable() )`
`3325`	`3325`	`return false;`
`3326`	`3326`
	`3327`	`+ if ( field.name().isEmpty() )`
	`3328`	`+ return false;`
	`3329`	`+`
`3327`	`3330`	`for ( QgsFieldMap::const_iterator it = mUpdatedFields.begin(); it != mUpdatedFields.end(); it++ )`
`3328`	`3331`	`{`
`3329`	`3332`	`if ( it.value().name() == field.name() )`