library: fix a flaw in one 'new' function, <diskstats>

At 'new' time, the major API modules each ensure their
'info' parameter isn't NULL but what it pointed to was
except this single straggler, for some unknown reason.

So, this patch brings him into line with those others.

[ And, without going into the ugly details, this was ]
[ the reason I never experienced an abend originally ]
[ but Craig did. And, though related to stacks mgmt, ]
[ zero initialization was not a factor. Anyway, with ]
[ this patch, everybody would have experienced abend ]
[ under the original (faulty) test_Itemtables logic! ]

Reference(s):
https://www.freelists.org/post/procps/keep-on-patchin,13

Signed-off-by: Jim Warner <james.warner@comcast.net>
This commit is contained in:
Jim Warner authored and Craig Small committed 2020-08-24 21:37:52 +10:00
1 parent 0cddb06425
commit 81db6664c9
1 file changed
+1 -1
+1 -1
View File
@@ -733,7 +733,7 @@ PROCPS_EXPORT int procps_diskstats_new (
if (failed) _Exit(EXIT_FAILURE);
#endif
if (info == NULL)
if (info == NULL || *info != NULL)
return -EINVAL;
if (!(p = calloc(1, sizeof(struct diskstats_info))))
return -ENOMEM;