mod_nss

#31 nss_engine_kernel.c:555]: (style) Suspicious condition
Closed: duplicate Opened by dcb.

Closed: duplicate
Reopen Issue

nss_engine_kernel.c:555]: (style) Suspicious condition (assignment + comparison); Clarify expression with parentheses.

Source code is

if ((result = ap_get_brigade(c->input_filters, bb,
AP_MODE_GETLINE, APR_BLOCK_READ, HUGE_STRING_LEN) !=
APR_SUCCESS || APR_BRIGADE_EMPTY(bb))) {

Maybe better code

if ((result = ap_get_brigade(c->input_filters, bb,
AP_MODE_GETLINE, APR_BLOCK_READ, HUGE_STRING_LEN)) !=
APR_SUCCESS || APR_BRIGADE_EMPTY(bb)) {

Duplicate of issue 28

Metadata Update from @rcritten:
- Issue close_status updated to: duplicate
- Issue priority set to: None (was: 4)
- Issue status updated to: Closed (was: Open)
Metadata
None
None
None
None
None
defect
core
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.