tree 067f75547ef0b6460811834357fb0f9c99e958a2
parent d9bae7096f30bc3cf27829de1f095df0bcaa7211
author Willy Tarreau <w@1wt.eu> 1725951329 +0200
committer Willy Tarreau <w@1wt.eu> 1725952597 +0200
encoding latin1

BUG/MINOR: pattern: do not leave a leading comma on "set" error messages

Commit 4f2493f355 ("BUG/MINOR: pattern: pat_ref_set: fix UAF reported by
coverity") dropped the condition to concatenate error messages and as
such introduced a leading comma in front of all of them. Then commit
911f4d93d4 ("BUG/MINOR: pattern: pat_ref_set: return 0 if err was found")
changed the behavior to stop at the first error anyway, so all the
mechanics dedicated to the concatenation of error messages is no longer
needed and we can simply return the error as-is, without inserting any
comma.

This should be backported where the patches above are backported.

(cherry picked from commit 9f8d9c9e8bdef8e7b8f7d1d641987b4d0dcae988)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit 6cae9b3ead0af8c751e9ed1e160432ae4a8773c6)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit ac9756e9e844d7abf4523e65d965dd11310e703f)
[wt: ctx adj]
Signed-off-by: Willy Tarreau <w@1wt.eu>