On Thu, December 10, 2015 8:53 am, Axel Beckert wrote:
Hi JC,
On Thu, Dec 10, 2015 at 08:23:23AM -0800, J.C. Cleaver wrote:
Can you see if this patch helps?
The patched helped, thanks! Can't reproduce the issue anymore.
Perfect, thanks!
-jc
(From earlier)
--- web/confreport.c (revision 7835)
+++ web/confreport.c (working copy)
@@ -288,9 +288,10 @@
}
else if (is_net_test(itm)) {
colname = strdup(itm);
+ visdata = strdup("");
}
• + if (!visdata) visdata = strdup("");
Those two additionans look a little bit redundant, but that shouldn't
cause any harm. (I'd say the first addition shouldn't be necessary
if the second one is present.)
That's correct. I added that primarily because of the structures above it
(which all set visdata), and because I wasn't quite sure of ultimate
source of some of this, and what was liable to fall through or not.