Thread (10 messages) 10 messages, 4 authors, 2022-11-15

Re: [PATCH v2] maintenance --unregister: fix uninit'd data use & -Wdeclaration-after-statement

From: Taylor Blau <hidden>
Date: 2022-11-15 18:54:51

On Tue, Nov 15, 2022 at 06:00:45PM +0000, ronan@rjp.ie wrote:
November 15, 2022 10:35 AM, "Taylor Blau" [off-list ref] wrote:
quoted
It was human error on my part. Please let me know if the result in
'next' now is OK to you.
Had a look and I'm just wondering if we're sure it's alright to clear
the configset before using the list we found in it. Doesn't that list
point into the configset?
It does. I should not have trusted the incoming patch so blindly. See:

    https://lore.kernel.org/git/2cbead254b77cb02d219bca8f628dc4362c045b0.1668538355.git.me@ttaylorr.com/ (local)

for a fixup on top.

Thanks,
Taylor
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help