Skip to content

Commit 127c290

Browse files
committed
Refactor AsyncApiWalker to ensure null checks before walking references
Added explicit null checks for `reply.Address` and `reply.Channel`. This prevents potential NullReferenceException when these properties are not initialized. This change ensures that the walker safely handles cases where optional fields might be missing, improving robustness of the code.
1 parent 714f546 commit 127c290

1 file changed

Lines changed: 10 additions & 3 deletions

File tree

src/ByteBard.AsyncAPI/Services/AsyncApiWalker.cs

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -587,8 +587,15 @@ private void Walk(AsyncApiOperationReply reply)
587587

588588
this.visitor.Visit(reply);
589589

590-
this.Walk(reply.Address);
591-
this.Walk(reply.Channel as IAsyncApiReferenceable);
590+
if (reply.Address != null)
591+
{
592+
this.Walk(reply.Address);
593+
}
594+
595+
if (reply.Channel != null)
596+
{
597+
this.Walk(reply.Channel as IAsyncApiReferenceable);
598+
}
592599

593600
foreach (var message in reply.Messages)
594601
{
@@ -1211,4 +1218,4 @@ public void Walk(IAsyncApiElement element)
12111218
}
12121219
}
12131220
}
1214-
}
1221+
}

0 commit comments

Comments
 (0)