Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 226001 - sys-auth/pam_mount-0.40 has broken digest (wrong filesize)
Summary: sys-auth/pam_mount-0.40 has broken digest (wrong filesize)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Hanno Böck
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-12 00:47 UTC by Randall Wald
Modified: 2008-06-13 08:29 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Randall Wald 2008-06-12 00:47:40 UTC
Sorry to bother you about a trivial bug, but the Manifest file for sys-auth/pam_mount-0.40 apparently has the wrong size and sums; it claims the pam_mount-0.40.tar.bz2 file should have 307548 bytes and sha1sum to 2b7531976cdbd4fcad387e16e79d87e7672203e0, but the version I downloaded (from http://osdn.dl.sourceforge.net/sourceforge/pam-mount/pam_mount-0.40.tar.bz2) is only 307208 bytes and has a sha1sum of cff664993b06c63fdbbdf3164081a7d46e5f5225. Anyway, the exact error is:

 * checking ebuild checksums ;-) ...                                     [ ok ]
 * checking auxfile checksums ;-) ...                                    [ ok ]
 * checking miscfile checksums ;-) ...                                   [ ok ]
 * checking pam_mount-0.40.tar.bz2 ;-) ...                               [ !! ]

!!! Digest verification failed:
!!! /usr/portage/distfiles/pam_mount-0.40.tar.bz2
!!! Reason: Filesize does not match recorded size
!!! Got: 307208
!!! Expected: 307548
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2008-06-12 03:12:15 UTC
Confirmed.

('Filesize does not match recorded size', 307208L, 307548)
!!! Fetched file: pam_mount-0.40.tar.bz2 VERIFY FAILED!
!!! Reason: Filesize does not match recorded size
!!! Got:      307208
!!! Expected: 307548
Refetching... File renamed to '/keeps/gentoo/distfiles/pam_mount-0.40.tar.bz2._checksum_failure_.79icQr'

>>> Downloading 'http://kent.dl.sourceforge.net/sourceforge/pam-mount/pam_mount-0.40.tar.bz2'
--2008-06-12 05:11:23--  http://kent.dl.sourceforge.net/sourceforge/pam-mount/pam_mount-0.40.tar.bz2
Resolving kent.dl.sourceforge.net... 212.219.56.167
Connecting to kent.dl.sourceforge.net|212.219.56.167|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 307208 (300K) [application/x-bzip2]
Saving to: `/keeps/gentoo/distfiles/pam_mount-0.40.tar.bz2'

100%[======================================>] 307,208      550K/s   in 0.5s

2008-06-12 05:11:25 (550 KB/s) - `/keeps/gentoo/distfiles/pam_mount-0.40.tar.bz2' saved [307208/307208]

('Filesize does not match recorded size', 307208L, 307548)
!!! Fetched file: pam_mount-0.40.tar.bz2 VERIFY FAILED!
!!! Reason: Filesize does not match recorded size
!!! Got:      307208
!!! Expected: 307548
Refetching... File renamed to '/keeps/gentoo/distfiles/pam_mount-0.40.tar.bz2._checksum_failure_.79icQr'
Comment 2 Hanno Böck gentoo-dev 2008-06-12 11:24:42 UTC
Thanks, upstream changed the tarball after the release. bumped to cover this.
Comment 3 Christian Schmidt 2008-06-12 11:56:37 UTC
Can you attach the old tarball to this bug, or verify if there are any changes to the content, and if yes, which?
Comment 4 Hanno Böck gentoo-dev 2008-06-12 12:05:18 UTC
diff -Naur 1/pam_mount-0.40/src/pam_mount.c 2/pam_mount-0.40/src/pam_mount.c
--- 1/pam_mount-0.40/src/pam_mount.c    2008-06-11 20:51:21.000000000 +0200
+++ 2/pam_mount-0.40/src/pam_mount.c    2008-06-11 21:14:19.000000000 +0200
@@ -287,7 +287,7 @@
                        goto out;
                }
                /* get password directly */
-               ret = read_password(pamh, "foo" /* Config.msg_authpw */, &authtok);
+               ret = read_password(pamh, Config.msg_authpw, &authtok);
                if (ret != PAM_SUCCESS) {
                        l0g("error trying to read password\n");
                        goto out;
@@ -506,7 +506,7 @@
                        goto out;
                }
                l0g("error trying to retrieve authtok from auth code\n");
-               ret = read_password(pamh, "yo" /* Config.msg_sessionpw */, &system_authtok);
+               ret = read_password(pamh, Config.msg_sessionpw, &system_authtok);
                if (ret != PAM_SUCCESS) {
                        l0g("error trying to read password\n");
                        goto out;
Comment 5 Christian Schmidt 2008-06-13 08:29:02 UTC
Thanks a lot, that change seems to make pretty much sense ;)