Correct the success return value for au_read_rec() in au_io.3

[aniketp]

Summary

The manpage of au_io.3 shows the incorrect success return value of au_read_rec() as 0,
since it returns the number of bytes read from a file stream.
Correct implementation done here: bsmtrace/bsm.c

 C
while ((reclen = au_read_rec(fp, &bsm_rec)) != -1) {
        ....
        bd.br_raw_len = reclen;
        ....
}

Here, on success, the function returns no. of bytes which is stored in reclen.

Pull Request: https://github.com/openbsm/openbsm/pull/25

Test Plan

Use au_read_rec() to correctly read the BSM tokens. On printing the return value,
you'd either see -1 if something went wrong or the length of the tokens read if the implementation is correct.

[asomers]

contrib/openbsm/libbsm/au_io.3

27 ↗

(On Diff #43143)

You need to bump the man page revision date.

132 ↗

(On Diff #43143)

You don't need "The function". It's ok to use function names like proper nouns.

134 ↗

(On Diff #43143)

s/returns 0/returns 0 on success/

136 ↗

(On Diff #43143)

The comma is unneeded. Though the sentence might actually sound better if you move "on success" to the beginning.

[aniketp]

Update the revision following the inline suggestions