Hello All,

I came across something that looks fishy in the source code for
|librecsort| in the AST sources -- on GitHub -- here:

https://github.com/att/ast/blob/c506cb548d9b4bcebef92c86e948657728760e15/src/lib/librecsort/rsmerge.c#L405

This is the source file 'src/lib/librecsort/rsmerge.c' at line 405.

The part of the line:

        vmopen(&vmdisc,Vmlast,0)

looks like a bug to me, but I do not really know what I am looking at
any longer so if someone could confirm (or refute) that this is a bug, I
would appreciate it.

The variable 'vmdisc' is itself a pointer to a Vmdisc_t object, but the
subroutine |vmopen| is supposed to take a pointer to a Vmalloc_t object
(if I remember correctly). So passing '&vmdisc' (address of pointer) to
|vmopen| looks very fishy (bad).

Is this a bug? And, if so what is the fix?  I do not remember a
patch on this already, but I could have missed it.

Thanks for any help or information on this.

----
David A.D. Morano
[email protected]

_______________________________________________
ast-users mailing list
[email protected]
http://lists.research.att.com/mailman/listinfo/ast-users

Reply via email to