Project

General

Profile

Actions
Copy link

Bug #5005

closed

libnvpair JSON output broken by lint fixes

Added by Joshua M. Clulow almost 8 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Joshua M. Clulow
Category:
lib - userland libraries
Start date:
2014-07-14
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

As part of the upstreaming of OS-1840 (i.e. #4304), some fixes were made to placate lint. Most of these were fine, but one of them removed a constant condition in a do {} while(0) loop being used as part of the FPRINTF macro – the removal of this surround in the macro made it unsafe for use in nested if else statements and thus the logic is entirely broken.

We should fix the damage and also introduce some tests into the test suite to ensure this does not happen again.

(Originally Joyent-internal bug OS-3068 )

Actions
Copy link

Also available in: Atom PDF