Stream fails when list contains fewer items than initialCount #8714

Activity

1. MikeBarker-MSFT

   added the

   Bug

   issue type on Sep 16, 2025
2. github-actions

   added

   🌶️ hot chocolate

   on Sep 16, 2025
3. MikeBarker-MSFT commented on Sep 16, 2025

   

   MikeBarker-MSFT

   on Sep 16, 2025

   ContributorAuthor

   More actions

   I believe the issue is caused by these lines in ResolverTask.Execute.cs

   while (next)
   {
       count++;
       next = await enumerator.MoveNextAsync().ConfigureAwait(false);
       list.Add(enumerator.Current);
   
       if (count >= initialCount)
       {
           break;
       }
   }

   Here, the result of MoveNextAsync is not tested before calling list.Add(enumerator.Current), and because of this null (or, more correctly the default value) is placed in the list. When this is later read, it causes a NullReferenceException to be thrown.
4. MikeBarker-MSFT commented on Sep 16, 2025

   

   MikeBarker-MSFT

   on Sep 16, 2025

   ContributorAuthor

   More actions

   I am going to pop up a PR with a proposed changeset.
5. michaelstaib

   closed this as completedin #8715on Sep 17, 2025
6. michaelstaib

   added 2 commits that reference this issue on Sep 17, 2025

   Fix #8714: @stream fails when list contains fewer items than initialC…

   ...

   This issue will close once commit 324039b is merged into the 'main' branch.

   324039b

   Fix ChilliCream#8714: @stream fails when list contains fewer items th…

   ...

   This issue will close once commit 3b80da6 is merged into the 'main' branch.

   3b80da6