--- a/service/SettingsConfuse.cpp
+++ b/service/SettingsConfuse.cpp
@@ -42,10 +42,10 @@ Settings::Settings(void) {
 */
 Settings::~Settings(void) {
 	TelldusCore::MutexLocker locker(&mutex);
-	if (d->cfg > 0) {
+	if (d->cfg) {
 		cfg_free(d->cfg);
 	}
-	if (d->var_cfg > 0) {
+	if (d->var_cfg) {
 		cfg_free(d->var_cfg);
 	}
 	delete d;
@@ -56,7 +56,7 @@ Settings::~Settings(void) {
 */
 std::wstring Settings::getSetting(const std::wstring &strName) const {
 	TelldusCore::MutexLocker locker(&mutex);
-	if (d->cfg > 0) {
+	if (d->cfg) {
 		std::string setting(cfg_getstr(d->cfg, TelldusCore::wideToString(strName).c_str()));
 		return TelldusCore::charToWstring(setting.c_str());
 	}
@@ -68,7 +68,7 @@ std::wstring Settings::getSetting(const
 */
 int Settings::getNumberOfNodes(Node node) const {
 	TelldusCore::MutexLocker locker(&mutex);
-	if (d->cfg > 0) {
+	if (d->cfg) {
 		if (node == Device) {
 			return cfg_size(d->cfg, "device");
 		} else if (node == Controller) {
