Sub-subclasses can access protected constructor

sandersn · sandersn · commit b321d50170cc · 2016-12-05T12:29:03.000-08:00
diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
@@ -13467,13 +13467,14 @@ namespace ts {
                 const containingClass = getContainingClass(node);
                 if (containingClass) {
                     const containingType = getTypeOfNode(containingClass);
-                    const baseTypes = getBaseTypes(<InterfaceType>containingType);
-                    if (baseTypes.length) {
+                    let baseTypes = getBaseTypes(containingType as InterfaceType);
+                    while (baseTypes.length) {
                         const baseType = baseTypes[0];
                         if (modifiers & ModifierFlags.Protected &&
                             baseType.symbol === declaration.parent.symbol) {
                             return true;
                         }
+                        baseTypes = getBaseTypes(baseType as InterfaceType);
                     }
                 }
                 if (modifiers & ModifierFlags.Private) {

Original file line number	Diff line number	Diff line change
`@@ -13467,13 +13467,14 @@ namespace ts {`
`13467`	`13467`	`const containingClass = getContainingClass(node);`
`13468`	`13468`	`if (containingClass) {`
`13469`	`13469`	`const containingType = getTypeOfNode(containingClass);`
`13470`		`- const baseTypes = getBaseTypes(<InterfaceType>containingType);`
`13471`		`- if (baseTypes.length) {`
	`13470`	`+ let baseTypes = getBaseTypes(containingType as InterfaceType);`
	`13471`	`+ while (baseTypes.length) {`
`13472`	`13472`	`const baseType = baseTypes[0];`
`13473`	`13473`	`if (modifiers & ModifierFlags.Protected &&`
`13474`	`13474`	`baseType.symbol === declaration.parent.symbol) {`
`13475`	`13475`	`return true;`
`13476`	`13476`	`}`
	`13477`	`+ baseTypes = getBaseTypes(baseType as InterfaceType);`
`13477`	`13478`	`}`
`13478`	`13479`	`}`
`13479`	`13480`	`if (modifiers & ModifierFlags.Private) {`