Fix #49399

src/vs/workbench/services/configuration/common/configurationExtensionPoint.ts

@@ -11,6 +11,7 @@ import { ExtensionsRegistry, IExtensionPointUser } from 'vs/workbench/services/e
 import { IConfigurationNode, IConfigurationRegistry, Extensions, editorConfigurationSchemaId, IDefaultConfigurationExtension, validateProperty, ConfigurationScope } from 'vs/platform/configuration/common/configurationRegistry';
 import { IJSONContributionRegistry, Extensions as JSONExtensions } from 'vs/platform/jsonschemas/common/jsonContributionRegistry';
 import { workspaceSettingsSchemaId, launchSchemaId } from 'vs/workbench/services/configuration/common/configuration';
+import { isObject } from 'vs/base/common/types';
 
 const configurationRegistry = Registry.as<IConfigurationRegistry>(Extensions.Configuration);
 
@@ -131,7 +132,17 @@ function validateProperties(configuration: IConfigurationNode, extension: IExten
 		}
 		for (let key in properties) {
 			const message = validateProperty(key);
+			if (message) {
+				delete properties[key];
+				extension.collector.warn(message);
+				continue;
+			}
 			const propertyConfiguration = configuration.properties[key];
+			if (!isObject(propertyConfiguration)) {
+				delete properties[key];
+				extension.collector.error(nls.localize('invalid.property', "'configuration.property' must be an object"));
+				continue;
+			}
 			if (propertyConfiguration.scope) {
 				if (propertyConfiguration.scope.toString() === 'application') {
 					propertyConfiguration.scope = ConfigurationScope.APPLICATION;
@@ -144,10 +155,6 @@ function validateProperties(configuration: IConfigurationNode, extension: IExten
 				propertyConfiguration.scope = ConfigurationScope.WINDOW;
 			}
 			propertyConfiguration.notMultiRootAdopted = !(extension.description.isBuiltin || (Array.isArray(extension.description.keywords) && extension.description.keywords.indexOf('multi-root ready') !== -1));
-			if (message) {
-				extension.collector.warn(message);
-				delete properties[key];
-			}
 		}
 	}
 	let subNodes = configuration.allOf;