libtransport.git Changeset - dd9f9c48e1ed

Changeset - dd9f9c48e1ed

Parent rev.

Child rev.

[Not reviewed]

0 1 0

Jan Kaluza - 13 years ago 2012-07-16 14:38:52
hanzz.k@gmail.com

Do not try to store buddies with empty name and clear m_buddies after storing

1 file changed with 8 insertions and 0 deletions:

src/rosterstorage.cpp

0 comments (0 inline, 0 general)

src/rosterstorage.cpp

➞

Show inline comments

 RosterStorage::~RosterStorage() {
 	m_storageTimer->stop();
+}
 void RosterStorage::storeBuddy(Buddy *buddy) {
 	if (!buddy) {
 		return;
+	}
 	if (buddy->getName().empty()) {
 		return;
+	}
 	m_buddies[buddy->getName()] = buddy;
 	m_storageTimer->start();
+}
 bool RosterStorage::storeBuddies() {
 	if (m_buddies.size() == 0) {
@@ @@ -126,12 +133,13 @@ bool RosterStorage::storeBuddies() { @@
 // 			s->id = id;
 // 			g_hash_table_foreach(buddy->node.settings, save_settings, s);
 // 			delete s;
 // 		}
+	}
 	m_buddies.clear();
 	m_storageBackend->commitTransaction();
 	return true;
+}
 void RosterStorage::removeBuddyFromQueue(Buddy *buddy) {
 	m_buddies.erase(buddy->getName());

0 comments (0 inline, 0 general)