Merge pull request #1089 from fales1488/patch-1

drewnoakes · web-flow · commit 173358cbb1b2 · 2024-05-09T07:33:00.000+10:00
Fix Division by Zero in FairQueueing when m_active is 0
diff --git a/src/NetMQ/Core/Patterns/Utils/FairQueueing.cs b/src/NetMQ/Core/Patterns/Utils/FairQueueing.cs
@@ -122,7 +122,7 @@ public bool RecvPipe(ref Msg msg, [NotNullWhen(returnValue: true)] out Pipe? pip
 
                     m_more = msg.HasMore;
                     if (!m_more)
-                        m_current = (m_current + 1) % m_active;
+                        m_current = m_active > 0 ? (m_current + 1) % m_active : 0;
                     return true;
                 }
 

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ public bool RecvPipe(ref Msg msg, [NotNullWhen(returnValue: true)] out Pipe? pip`
`122`	`122`
`123`	`123`	`m_more = msg.HasMore;`
`124`	`124`	`if (!m_more)`
`125`		`- m_current = (m_current + 1) % m_active;`
	`125`	`+ m_current = m_active > 0 ? (m_current + 1) % m_active : 0;`
`126`	`126`	`return true;`
`127`	`127`	`}`
`128`	`128`