rianjs / ical.net

ical.NET - an open source iCal library for .NET
MIT License
782 stars 230 forks source link

Fixed potential stack overflow when calculating changes to DtStart, DtEnd, etc. #473

Closed Carbonitex closed 3 years ago

Carbonitex commented 4 years ago

These changes here came from our copy of ical.net. Figured it would be good to push these potentially back. We were seeing some production systems getting stack overflows caused by some fields not being provided when creating a calendar event then setting a value. After setting this value ExtrapolateTimes would then start looping when attempting to set other DtStart, Duration, DtEnd values.

p.s This is my first pull request, familiar with using open source projects but this is my first time going upstream. If there are any changes or suggestions let me know.