Skip to content

Commit 860f2cb

Browse files
antonwolfyclaude
andcommitted
fix: silence Coverity UNUSED_VALUE in __create_descriptor_1d
The value returned by DftiFreeDescriptor on the reallocate path is consumed only by a debug-only assert(status == 0). Under NDEBUG the assert is stripped, so the store is dead before the unconditional status = 0 reset, which Coverity flags as UNUSED_VALUE (one CID per .c.src template instantiation). Mark the value as intentionally unused with (void)status to make the intent explicit and clear the finding. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 25ceb1e commit 860f2cb

2 files changed

Lines changed: 2 additions & 0 deletions

File tree

‎CHANGELOG.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1111
### Changed
1212

1313
### Fixed
14+
* Silenced a Coverity `UNUSED_VALUE` finding in `__create_descriptor_1d` by marking the `DftiFreeDescriptor` status (used only by a debug-only `assert`) as intentionally unused
1415

1516
## [2.3.2] - 2026-08-04
1617

‎mkl_fft/src/mklfft.c.src‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,7 @@ __create_descriptor_1d_@prec@_@domain@(MKL_LONG len, @sc_t@ fsc, @sc_t@ bsc, Dft
170170
if(dfti_cache->hand) {
171171
status = DftiFreeDescriptor(&(dfti_cache->hand));
172172
assert(status == 0);
173+
(void)status; /* used only by assert; ignored under NDEBUG */
173174
}
174175

175176
status = 0;

0 commit comments

Comments
 (0)