[ofw] [bug] Assert in opensm 3.3.11

[Smith, Stan]

Hello,
  I too noticed this change, no assert() fire on win7; not surprising.  Seems the discussion Alex, Hal and others had w.r.t. Windows setvbuf(BUFSIZ) did not make it into the mainline.
I'd prefer BUFSIZ as it's a well-recognized system wide entity, although 1024 would work equally well.
I'll update, thanks for pointing out the omission.

Stan.



From: Leonid Keller [mailto:leonid at mellanox.co.il]
Sent: Saturday, October 15, 2011 11:23 AM
To: Smith, Stan
Cc: ofw_list
Subject: [ofw][bug] Assert in opensm 3.3.11

Hi Stan,

I got an assert in Win8 on setvbuf (see below).
It states that size should be at least 2.
Also MSDN agrees with that:
size
Buffer size in bytes. Allowable range: 2 <= size <= INT_MAX (2147483647)
Here is the suggested patch.

Index: main.c
===================================================================
--- main.c             (revision 8962)
+++ main.c          (revision 8963)
@@ -732,7 +732,7 @@
                };

                /* force stdout to be line-buffered */
-              setvbuf(stdout, NULL, _IOLBF, 0);
+             setvbuf(stdout, NULL, _IOLBF, 1024);

                /* Make sure that the opensm and complib were compiled using
                   same modes (debug/free) */
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20111017/30e93840/attachment.html>