Patch - FieldList Crash

Hi,

 Attached is a patch for kexidb/fieldlist.cpp. Earlier, if 
FieldList::autoIncrementFields()  ( which, doxygen tells me,
is not called by 
anyone else till now ), was responsible for a crash in the
future when the 
FieldList object was being destroyed . 

There was an attempt to delete the autoincrement fields two
times. The first 
time is when the members of  m_autoinc_fields are deleted,
and the second 
time is when members of FieldList::m_fields are deleted.

Any suggestions ?

Cheers!
 Sharan Rao


_______________________________________________
Kexi mailing list
Kexikde.org
https://ma
il.kde.org/mailman/listinfo/kexi

Sharan Rao said the following, On 2007-12-30 23:51:
> Hi,
> 
>  Attached is a patch for kexidb/fieldlist.cpp. Earlier,
if 
> FieldList::autoIncrementFields()  ( which, doxygen
tells me, is not called by 
> anyone else till now ), was responsible for a crash in
the future when the 
> FieldList object was being destroyed . 
> 
> There was an attempt to delete the autoincrement fields
two times. The first 
> time is when the members of  m_autoinc_fields are
deleted, and the second 
> time is when members of FieldList::m_fields are
deleted.
> 

OK, thank!
please commit

I'll consider making KexiDB::Field::List not-autodeleted by
default to avoid
such misuses.

-- 
regards / pozdrawiam, Jaroslaw Staniek
  Sponsored by OpenOffice Polska (http://www.openoffice
.com.pl/en) to work on
  Kexi & KOffice (http://www.kexi.pl/en, http://www.koffice.org/ke
xi)
  KDE Libraries for MS Windows (http://windows.kde.org)

_______________________________________________
Kexi mailing list
Kexikde.org
https://ma
il.kde.org/mailman/listinfo/kexi