From: Zdenek S. <zde...@gm...> - 2013-08-29 07:58:44
|
On Thu, Aug 29, 2013 at 9:37 AM, Liebig, Holger <hol...@ts...> wrote: > Jim, > any chance to include my small patch from 2011 regarding SOL input buffer size? > The problem is related when a user is using copy&paste and is pasting large blocks of text (>255) into the ipmitool SOL session at once. Ipmitool is using the reported SOL max inbound payload size for the allocated data buffer size, but is not accounting for the standard 4 byte SOL header which is part of the payload. So, when the BMC is reporting a SOL payload size of 259 (255 max ACK'able characters/bytes + 4byte for the header) ipmitool is using a buffer of 259 characters for text to send at once. > > The problem was reported by a customer and the fix has been integrated into Red Hat's version of ipmitool. > > Problem description: > https://access.redhat.com/site/solutions/63758 > > Solution (need an account) > https://bugzilla.redhat.com/show_bug.cgi?id=675980 > > Thanks, > Holger > Holger, release of 1.8.13 was announced well ahead, there was plenty of time etc. etc. No, no more fixes into 1.8.13. It's 28th August and we could do this forever - until all bugs and problems are fixed and everybody is satisfied which is never. Thanks for understanding, Z. > > -----Original Message----- > From: Jim Mankovich [mailto:jm...@hp...] > Sent: Monday, August 26, 2013 7:25 PM > To: ipm...@li... > Subject: [Ipmitool-devel] ipmitool 1.8.13rc1 release is available > > All, > > I uploaded an updated preliminary release of ipmitool 1.8.13 for testing to the source forge ipmitool files repository. > > https://sourceforge.net/projects/ipmitool/files/ipmitool/1.8.13rc1/ > > The 1.8.13rc1 release currently contains all the changes we had planned to include into 1.8.13, so if there are no defects filed against > 1.8.13rc0 or 1.8.13rc1, what is in rc1 will become the released bits for > 1.8.13 in early September. > > Thanks, > Jim > > -- > --- Jim Mankovich | jm...@hp... (US Mountain Time) --- > > > ------------------------------------------------------------------------------ > Introducing Performance Central, a new site from SourceForge and > AppDynamics. Performance Central is your source for news, insights, > analysis and resources for efficient Application Performance Management. > Visit us today! > http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk > _______________________________________________ > Ipmitool-devel mailing list > Ipm...@li... > https://lists.sourceforge.net/lists/listinfo/ipmitool-devel > > > ---------- Forwarded message ---------- > From: "Liebig, Holger" <hol...@ts...> > To: "ipm...@li..." <ipm...@li...> > Cc: Jan Safranek <jsa...@re...> > Date: Tue, 8 Feb 2011 14:08:04 +0100 > Subject: [Ipmitool-devel] [PATCH] SOL input buffer size > Hi, > The following patch fixes an issue where ipmitool is potentially sending to much input data to the BMC with a single SOL packet. Depending on the BMC's implementation, this can lead to multiple echo's of the characters at the end, or an endless retry loop from the BMC. > > Best Regards, > Holger Liebig > > diff -Nur ipmitool.sav/lib/ipmi_sol.c ipmitool/lib/ipmi_sol.c > --- ipmitool.sav/lib/ipmi_sol.c 2009-01-18 14:53:12.000000000 +0100 > +++ ipmitool/lib/ipmi_sol.c 2011-02-08 13:45:51.574399671 +0100 > @@ -1537,6 +1537,10 @@ > int keepAliveRet = 0; > int retrySol = 0; > > + /* Subtract SOL header from max_inbound_payload_size */ > + if (buffer_size > 4) > + buffer_size -= 4; > + > buffer = (char*)malloc(buffer_size); > if (buffer == NULL) { > lprintf(LOG_ERR, "ipmitool: malloc failure"); > > ------------------------------------------------------------------------------ > Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more! > Discover the easy way to master current and previous Microsoft technologies > and advance your career. Get an incredible 1,500+ hours of step-by-step > tutorial videos with LearnDevNow. Subscribe today and save! > http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk > _______________________________________________ > Ipmitool-devel mailing list > Ipm...@li... > https://lists.sourceforge.net/lists/listinfo/ipmitool-devel > |