Date: Fri, 1 Jun 2007 12:27:11 -0400 (EDT) From: "Mikhail T." <mi@aldan.algebra.com> To: FreeBSD-gnats-submit@FreeBSD.org Subject: bin/113230: const-ify PAM-headers Message-ID: <200706011627.l51GRBqb091904@aldan.algebra.com> Resent-Message-ID: <200706011650.l51Go5ek045701@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 113230 >Category: bin >Synopsis: const-ify PAM-headers >Confidential: no >Severity: non-critical >Priority: medium >Responsible: freebsd-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: update >Submitter-Id: current-users >Arrival-Date: Fri Jun 01 16:50:04 GMT 2007 >Closed-Date: >Last-Modified: >Originator: Mikhail T. >Release: FreeBSD 6.2-STABLE amd64 >Organization: Virtual Estates, Inc. >Environment: System: FreeBSD aldan.algebra.com 6.2-STABLE FreeBSD 6.2-STABLE #2: Mon Mar 26 18:05:02 EDT 2007 mi@aldan.algebra.com:/meow/obj/var/src/sys/SILVER amd64 >Description: As currently declared, the pam_* functions don't guarantee the read-onlyness of the passed in data, even where no modifications could happen. The declarations also differ from those used of Solaris and Linux, leading to increased porting burden of 3rd-party PAM-modules (such as Samba's winbind). It is possible, that a newer version of OpenPAM will bring in these changes, but we can update the headers (without functional changes) now to introduce the updated API and ease the porting efforts. >How-To-Repeat: >Fix: The up-to-date patch can be found at http://aldan.algebra.com/~mi/pam-const.patch or -- color-coded -- at: http://aldan.algebra.com/~mi/pam-const.patch.html It does not, at the moment, touch the relevant man-pages. These will be updated soon, but testing/review of the code can begin now. The changes only add const-ness here and there -- no functional changes. >Release-Note: >Audit-Trail: >Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200706011627.l51GRBqb091904>