[Bug 909485] Review Request: memstomp - checker for passing overlapping memory arguments to mem* and str* functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=909485

--- Comment #4 from Jeff Law <law@xxxxxxxxxx> ---
memstomp's functionality is entirely development related.  ie, it makes no
sense to break out a -devel package merely as a container for an unversioned
.so.  So, I agree this is OK.

I changed the %define to %global.  I wasn't aware of the distinction, I'll keep
it in mind for the future.

The dependency on libz comes in via the libbfd reference as far as I can tell. 
memstomp doesn't use libz directly.

The call to exit is purposeful as memstomp will intercept exit calls so that it
can summarize its findings.  At least that's what the design looks like.  Right
now the summary itself is just stubbed.

WRT the version #s.  Yea, it'd be best if upstream made an official release. 
Arguably we should remove the git hash from the version # since it just
complicates NVR comparisons.  I'd already done that locally.

I'll also fix the behaviour -> behavior warning :-)

There's a man page on the way as well which should take care of that warning.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=OYHOG84utb&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]