Skip to content
This repository has been archived by the owner on Jan 6, 2022. It is now read-only.

Maximum depth of container nesting is not checked #9

Closed
guelfey opened this issue Feb 2, 2013 · 1 comment
Closed

Maximum depth of container nesting is not checked #9

guelfey opened this issue Feb 2, 2013 · 1 comment
Assignees
@guelfey
Labels
spec

Comments

@guelfey
Copy link
Owner

guelfey commented Feb 2, 2013

From the spec:

Use of variants may not cause a total message depth to be larger than 64, including other container types such as structures.

...

The maximum depth of container type nesting is 32 array type codes and 32 open parentheses.
@ghost ghost assigned guelfey Feb 2, 2013
@guelfey guelfey mentioned this issue Feb 2, 2013
Closed
guelfey pushed a commit that referenced this issue Feb 2, 2013
Check for depth of container types in signatures
5709be1 
Updates #9, but doesn't quite fix it (because variants may increase the depth,
but it isn't known whether they do unless the value is looked at, which is kind
of the whole point of variants).
@guelfey guelfey mentioned this issue Feb 2, 2013
Closed
@guelfey guelfey closed this as completed in b375f1a Feb 3, 2013
@guelfey
Copy link
Owner Author

guelfey commented Feb 3, 2013

This still has to be done for outgoing messages.

@guelfey guelfey reopened this Feb 3, 2013
@guelfey guelfey closed this as completed in 206e053 Feb 6, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@guelfey guelfey

Labels
spec
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@guelfey