Date: Tue, 7 Mar 2017 20:33:18 -0800 (PST) From: "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net> To: Dexuan Cui <dexuan@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r314891 - head/sys/boot/efi/loader Message-ID: <201703080433.v284XICJ098481@pdx.rh.CN85.dnsmgr.net> In-Reply-To: <201703080250.v282oTDF096183@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[ Charset UTF-8 unsupported, converting... ] > Author: dexuan > Date: Wed Mar 8 02:50:29 2017 > New Revision: 314891 > URL: https://svnweb.freebsd.org/changeset/base/314891 > > Log: > loader.efi: finally fix the off-by-one bug in efi_verify_staging_size() > > r314828(loader.efi: fix an off-by-one bug in efi_verify_staging_size()) > doesn't really fix the bug and this patch adds the missing part. > > It's a shame that I didn't make everything correct at the very beginning... My thoughts on that: Try as hard as we can and you well come up short on making software correct in the beginning as it is an iterative process. Be glad you have found and fixed yet another small problem, and hope that in doing so you have not introduced another. I thank you for your persuit of this latent issue in loader.efi! -- Rod Grimes rgrimes@freebsd.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201703080433.v284XICJ098481>