summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRemko Tronçon <git@el-tramo.be>2011-08-28 11:15:51 (GMT)
committerRemko Tronçon <git@el-tramo.be>2011-08-28 11:57:00 (GMT)
commit33af107d8d4ffdeada9ea964fe696af4a23c5b95 (patch)
tree8b471b7626dec86888fc9237a37c752714a75692 /Swiften/VCards
parent2faca4242e4de2568eb917df83fd1b9c21f33897 (diff)
downloadswift-contrib-33af107d8d4ffdeada9ea964fe696af4a23c5b95.zip
swift-contrib-33af107d8d4ffdeada9ea964fe696af4a23c5b95.tar.bz2
Catch boost file system errors when reading VCards.
Release-Notes: Catch errors when writing VCards to disk. Resolves: #897
Diffstat (limited to 'Swiften/VCards')
-rw-r--r--Swiften/VCards/VCardFileStorage.cpp56
1 files changed, 34 insertions, 22 deletions
diff --git a/Swiften/VCards/VCardFileStorage.cpp b/Swiften/VCards/VCardFileStorage.cpp
index a246838..cd5cc9c 100644
--- a/Swiften/VCards/VCardFileStorage.cpp
+++ b/Swiften/VCards/VCardFileStorage.cpp
@@ -48,41 +48,53 @@ VCardFileStorage::VCardFileStorage(boost::filesystem::path dir) : vcardsPath(dir
}
boost::shared_ptr<VCard> VCardFileStorage::getVCard(const JID& jid) const {
- boost::filesystem::path vcardPath(getVCardPath(jid));
- if (boost::filesystem::exists(vcardPath)) {
- ByteArray data;
- data.readFromFile(vcardPath.string());
+ try {
+ boost::filesystem::path vcardPath(getVCardPath(jid));
+ if (boost::filesystem::exists(vcardPath)) {
+ ByteArray data;
+ data.readFromFile(vcardPath.string());
- VCardParser parser;
- PayloadParserTester tester(&parser);
- tester.parse(data.toString());
- return boost::dynamic_pointer_cast<VCard>(parser.getPayload());
+ VCardParser parser;
+ PayloadParserTester tester(&parser);
+ tester.parse(data.toString());
+ return boost::dynamic_pointer_cast<VCard>(parser.getPayload());
+ }
+ else {
+ return boost::shared_ptr<VCard>();
+ }
}
- else {
+ catch (const boost::filesystem::filesystem_error& e) {
+ std::cerr << "ERROR: " << e.what() << std::endl;
return boost::shared_ptr<VCard>();
}
}
void VCardFileStorage::setVCard(const JID& jid, VCard::ref v) {
- boost::filesystem::path vcardPath(getVCardPath(jid));
- if (!boost::filesystem::exists(vcardPath.parent_path())) {
- try {
+ try {
+ boost::filesystem::path vcardPath(getVCardPath(jid));
+ if (!boost::filesystem::exists(vcardPath.parent_path())) {
boost::filesystem::create_directories(vcardPath.parent_path());
}
- catch (const boost::filesystem::filesystem_error& e) {
- std::cerr << "ERROR: " << e.what() << std::endl;
- }
+ boost::filesystem::ofstream file(getVCardPath(jid));
+ file << VCardSerializer().serializePayload(v);
+ file.close();
+ getAndUpdatePhotoHash(jid, v);
+ }
+ catch (const boost::filesystem::filesystem_error& e) {
+ std::cerr << "ERROR: " << e.what() << std::endl;
}
- boost::filesystem::ofstream file(getVCardPath(jid));
- file << VCardSerializer().serializePayload(v);
- file.close();
- getAndUpdatePhotoHash(jid, v);
}
boost::filesystem::path VCardFileStorage::getVCardPath(const JID& jid) const {
- std::string file(jid.toString());
- String::replaceAll(file, '/', "%2f");
- return boost::filesystem::path(vcardsPath / (file + ".xml"));
+ try {
+ std::string file(jid.toString());
+ String::replaceAll(file, '/', "%2f");
+ return boost::filesystem::path(vcardsPath / (file + ".xml"));
+ }
+ catch (const boost::filesystem::filesystem_error& e) {
+ std::cerr << "ERROR: " << e.what() << std::endl;
+ return boost::filesystem::path();
+ }
}
std::string VCardFileStorage::getPhotoHash(const JID& jid) const {