Le 9 nov. 2015 1:00 PM, "Paulo Flabiano Smorigo" <[email protected]> a écrit : > > On Mon, Nov 9, 2015 at 8:46 AM, Vladimir 'φ-coder/phcoder' Serbinenko > <[email protected]> wrote: > > On 09.11.2015 02:33, Paulo Flabiano Smorigo wrote: > >> + /* 64 entries should be enough */ > >> + table_size = sizeof (grub_uint64_t) * 64; > > Can we do something better than assuming a particular upper limit? You > > don't even pass this limit to the function in any way. You basically get > > a buffer overflow. Should we perhaps pass the limit in nentries? Or do > > something similar? > > For sas there is an input attribute called max that I use as an upper > limit. AFAIK, vscsi method doesn't have it. nentries is a output > attribute. > > I'll investigate it. > Can't we just properly free returned memory? Or perhaps even just tolerate the leak and only ensure that it happens only once per controller per GRUB run? > > > > > > _______________________________________________ > > Grub-devel mailing list > > [email protected] > > https://lists.gnu.org/mailman/listinfo/grub-devel > > > > _______________________________________________ > Grub-devel mailing list > [email protected] > https://lists.gnu.org/mailman/listinfo/grub-devel
_______________________________________________ Grub-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/grub-devel
