Bug 1222719 - rtld-audit man example code uses wrong format for cookies
Summary: rtld-audit man example code uses wrong format for cookies
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: man-pages
Version: rawhide
Hardware: All
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Jan Chaloupka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 975551 1222720
TreeView+ depends on / blocked
 
Reported: 2015-05-19 00:10 UTC by Ben Woodard
Modified: 2015-12-04 19:54 UTC (History)
2 users (show)

Fixed In Version: man-pages-3.53-6.fc20
Clone Of:
: 1222720 (view as bug list)
Environment:
Last Closed: 2015-05-30 15:40:16 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch to fix the problem. (2.26 KB, patch)
2015-05-19 00:10 UTC, Ben Woodard
no flags Details | Diff

Description Ben Woodard 2015-05-19 00:10:37 UTC
Created attachment 1026902 [details]
Patch to fix the problem.

Description of problem:

Here is a patch to the rtld-audit man page. It is really simple and trivial. In the example code you used %x rather than %p in the example code for an audit library. The problem is that it truncates the pointers on 64b platforms. So you get something like:

la_symbind64(): symname = strrchr sym->st_value = 0x7f4b8a3f8960
	ndx = 222 flags = 0x0 refcook = 8b53e5c8 defcook = 8b537e30

rather than:

la_symbind64(): symname = fclose sym->st_value = 0x7fa452dd49b0
	ndx = 1135 flags = 0x0 refcook = 0x7fa453f395c8 defcook = 0x7fa453f32e30

This has bit me a handful of times when playing around with audit test libraries to investigate its behavior.

Version-Release number of selected component (if applicable):
all

Comment 1 Ben Woodard 2015-05-19 00:11:38 UTC
I sent this to mtk.manpages to get it fixed upstream.

Comment 2 Jan Chaloupka 2015-05-19 13:02:51 UTC
Hi Ben,

thanks for reporting this issue to upstream. Do you have a message id for your email? I can not find it on linux-man.org ML so far. It can take 1-2 days before it propagates on the ML.

Jan

Comment 3 Ben Woodard 2015-05-19 15:32:49 UTC
I sent it off to Michael Kerrisk directly and got back this email today. Hadn't submitted a patch to linux-man before and wasn't familiar with that process. Michael seemed to be the maintainer and so I sent it to him.


From: "Michael Kerrisk (man-pages)" <mtk.manpages>
Subject: Re: patch for rtld-audit man page
Date: May 18, 2015 at 11:50:37 PM PDT
To: Ben Woodard <woodard>
Cc: mtk.manpages, "Carlos O'Donell" <codonell>

Hi Ben,

On 05/19/2015 02:05 AM, Ben Woodard wrote:
> Here is a patch to the rtld-audit man page. It is really simple and trivial. In 
> the example code you used %x rather than %p in the example code for an audit 
> library. The problem is that it truncates the pointers on 64b platforms. So you 
> get something like:
> 
> la_symbind64(): symname = strrchr sym->st_value = 0x7f4b8a3f8960
> ndx = 222 flags = 0x0 refcook = 8b53e5c8 defcook = 8b537e30
> 
> rather than:
> 
> la_symbind64(): symname = fclose sym->st_value = 0x7fa452dd49b0
> ndx = 1135 flags = 0x0 refcook = 0x7fa453f395c8 defcook = 0x7fa453f32e30
> 
> This has bit me a handful of times when playing around with audit test libraries 
> to investigate its behavior.

Thanks! Applied. 

Cheers,

Michael

Comment 4 Jan Chaloupka 2015-05-20 08:22:10 UTC
Thanks for his reply.

If you don't mind I will wait for another release of man-pages instead of patching the current one. It is quite often.

Is this issue relevant for f22 and lower?

Comment 5 Ben Woodard 2015-05-21 21:46:36 UTC
Yes it is relevant for F22 and lower.
Change it when you like but it would be good to get it fixed rather than just having it languish.

Comment 7 Fedora Update System 2015-05-22 10:11:17 UTC
man-pages-3.81-2.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/man-pages-3.81-2.fc22

Comment 8 Fedora Update System 2015-05-22 10:13:20 UTC
man-pages-3.69-3.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/man-pages-3.69-3.fc21

Comment 9 Fedora Update System 2015-05-22 10:15:30 UTC
man-pages-3.53-5.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/man-pages-3.53-5.fc20

Comment 10 Fedora Update System 2015-05-26 03:20:13 UTC
Package man-pages-3.69-3.fc21:
* should fix your issue,
* was pushed to the Fedora 21 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing man-pages-3.69-3.fc21'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2015-8755/man-pages-3.69-3.fc21
then log in and leave karma (feedback).

Comment 11 Fedora Update System 2015-05-29 19:45:43 UTC
man-pages-3.53-6.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/man-pages-3.53-6.fc20

Comment 12 Fedora Update System 2015-05-30 15:40:16 UTC
man-pages-3.69-3.fc21 has been pushed to the Fedora 21 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 13 Fedora Update System 2015-05-30 15:40:57 UTC
man-pages-3.81-2.fc22 has been pushed to the Fedora 22 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2015-06-10 19:08:17 UTC
man-pages-3.53-6.fc20 has been pushed to the Fedora 20 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.