[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Conclusion of yet another expensive UI folly (was: [PULL 00/21] NBD
From: |
Eric Blake |
Subject: |
Re: Conclusion of yet another expensive UI folly (was: [PULL 00/21] NBD and miscellaneous patches for 2023-06-01) |
Date: |
Fri, 2 Jun 2023 07:29:38 -0500 |
User-agent: |
NeoMutt/20230517 |
On Fri, Jun 02, 2023 at 08:32:34AM +0200, Markus Armbruster wrote:
> Pattern:
>
> First, one of us gets a bright idea on user-friendly interface (here:
> fractional sizes like 1.5M). Objections, if any, get brushed aside.
>
> Then the thing sprouts warts, tentacles, sores, and starts to give off
> that sickly-sweet smell of bugs feasting on misguided ideas.
>
> Until one of us spends a lot more time on containing and reducing the
> damage than the thing could ever be worth.
>
> Cobbler, stick to your last.
>
> Thanks, Eric!
The scary part was that I found several other bugs completely
unrelated to the read-out-of-bounds, merely by increasing unit test
coverage. When Hanna first pointed me to the problem asking if a
5-line patch would work, I was not expecting it to blow up into a
series touching more than 1000 lines (true, most of those lines were
in the unit tests).
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [PULL 06/21] test-cutils: Test more integer corner cases, (continued)
- [PULL 06/21] test-cutils: Test more integer corner cases, Eric Blake, 2023/06/01
- [PULL 14/21] cutils: Allow NULL str in qemu_strtosz, Eric Blake, 2023/06/01
- [PULL 19/21] cutils: Use parse_uint in qemu_strtosz for negative rejection, Eric Blake, 2023/06/01
- [PULL 13/21] test-cutils: Refactor qemu_strtosz tests for less boilerplate, Eric Blake, 2023/06/01
- [PULL 12/21] test-cutils: Prepare for upcoming semantic change in qemu_strtosz, Eric Blake, 2023/06/01
- [PULL 03/21] test-cutils: Avoid g_assert in unit tests, Eric Blake, 2023/06/01
- Re: [PULL 00/21] NBD and miscellaneous patches for 2023-06-01, Richard Henderson, 2023/06/01
- Conclusion of yet another expensive UI folly (was: [PULL 00/21] NBD and miscellaneous patches for 2023-06-01), Markus Armbruster, 2023/06/02
- Re: Conclusion of yet another expensive UI folly (was: [PULL 00/21] NBD and miscellaneous patches for 2023-06-01),
Eric Blake <=