On Tue, Sep 18, 2018 at 03:12:11PM +0200, Johannes Berg wrote: > From: Johannes Berg <johannes.b...@intel.com> > > In one of my previous patches in this area I introduced code > to pass out just the error message to store in the extack, for > use in NLA_REJECT. > > Change this code now to set both the error message and the bad > attribute pointer, and carry around a boolean indicating that > the values have been set. > > This will be used in the next patch to allow recursive validation > of nested policies, while preserving the innermost error message > rather than overwriting it with a generic out-level message.
Did you consider indicating the message level, and only overwrite the message that is already in there if the new message level is higher than the current one? This way the first to set an Error message will have it, and Warning messages would be overwritten by such if needed. But it also would cause the first warning to be held, and not the last one, as it does today. We want the first error, but the last warning otherwise. It would not be possible to overwrite if new_msglvl >= cur_msglvl because then it would trigger the initial issue again, so some extra logic would be needed to solve this. Marcelo
