comparison pidgin/gtkconv-theme-loader.c @ 32649:1caf13175039

Be a bit stricter about types in Info.plist.
author Elliott Sales de Andrade <qulogic@pidgin.im>
date Mon, 19 Sep 2011 20:36:05 +0000
parents a56483e407d4
children 8716216050ee
comparison
equal deleted inserted replaced
32648:a56483e407d4 32649:1caf13175039
132 return NULL; 132 return NULL;
133 } 133 }
134 134
135 /* Check for required keys: CFBundleName */ 135 /* Check for required keys: CFBundleName */
136 val = g_hash_table_lookup(info, "CFBundleName"); 136 val = g_hash_table_lookup(info, "CFBundleName");
137 if (!val) { 137 if (!val || !G_VALUE_HOLDS_STRING(val)) {
138 purple_debug_error("gtkconv-theme-loader", 138 purple_debug_error("gtkconv-theme-loader",
139 "%s/Contents/Info.plist missing required key CFBundleName.\n", 139 "%s/Contents/Info.plist missing required string key CFBundleName.\n",
140 dir); 140 dir);
141 g_hash_table_destroy(info); 141 g_hash_table_destroy(info);
142 return NULL; 142 return NULL;
143 } 143 }
144 CFBundleName = g_value_get_string(val); 144 CFBundleName = g_value_get_string(val);
145 145
146 /* Check for required keys: CFBundleIdentifier */ 146 /* Check for required keys: CFBundleIdentifier */
147 val = g_hash_table_lookup(info, "CFBundleIdentifier"); 147 val = g_hash_table_lookup(info, "CFBundleIdentifier");
148 if (!val) { 148 if (!val || !G_VALUE_HOLDS_STRING(val)) {
149 purple_debug_error("gtkconv-theme-loader", 149 purple_debug_error("gtkconv-theme-loader",
150 "%s/Contents/Info.plist missing required key CFBundleIdentifier.\n", 150 "%s/Contents/Info.plist missing required string key CFBundleIdentifier.\n",
151 dir); 151 dir);
152 g_hash_table_destroy(info); 152 g_hash_table_destroy(info);
153 return NULL; 153 return NULL;
154 } 154 }
155 CFBundleIdentifier = g_value_get_string(val); 155 CFBundleIdentifier = g_value_get_string(val);
156 156
157 /* Check for required keys: MessageViewVersion */ 157 /* Check for required keys: MessageViewVersion */
158 val = g_hash_table_lookup(info, "MessageViewVersion"); 158 val = g_hash_table_lookup(info, "MessageViewVersion");
159 if (!val) { 159 if (!val || !G_VALUE_HOLDS_INT(val)) {
160 purple_debug_error("gtkconv-theme-loader", 160 purple_debug_error("gtkconv-theme-loader",
161 "%s/Contents/Info.plist missing required key MessageViewVersion.\n", 161 "%s/Contents/Info.plist missing required integer key MessageViewVersion.\n",
162 dir); 162 dir);
163 g_hash_table_destroy(info); 163 g_hash_table_destroy(info);
164 return NULL; 164 return NULL;
165 } 165 }
166 166