1
82%
main: 82%

Ran 27 May 2026 07:08AM UTC

Files 64

Run time 6s

Badge

Embed ▾

Committed 26 May 2026 03:29PM UTC coverage: 73.069% (-7.6%) from 80.674%

Job # 26493872893.1

Build Type

push

github

Committed by

web-flow

Commit Message

(2.14) fix: JetStream consumer lock leak on start sequence error (#8230)

## Summary

This PR fixes a lock leak in JetStream consumer creation.

In `addConsumerWithAssignment`, `mset.mu` is held while creating a
consumer. In the direct/standalone path, if `o.selectStartingSeqNo()`
returns an error, the function currently returns without releasing
`mset.mu`.

This adds the missing `mset.mu.Unlock()` before returning the error.

Resolves #8229 

## Impact

Without this unlock, the stream mutex can remain locked after a starting
sequence error, which may cause later operations on the same stream to
block.

## Changes

- Add the missing `mset.mu.Unlock()` before returning from the
`selectStartingSeqNo()` error path.

## Detection

This issue was reported by
[`goconcurrencylint`](https://github.com/sanbricio/goconcurrencylint).

## Testing

- Not added yet. This is a minimal error-path fix identified by static
analysis.

Signed-off-by: Santiago Bricio <sanbriciorojas11@gmail.com>

Coverage Stats

68275 of 93439 relevant lines covered (73.07%)

488352.47 hits per line

nats-io / nats-server / 26493872893 / 1
82%
main: 82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 26493872893.1

nats-io / nats-server / 26493872893 / 1 82% main: 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 26493872893.1

nats-io / nats-server / 26493872893 / 1
82%
main: 82%

README BADGES
x