Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 677200 - dev-libs/cyrus-sasl USE=gdbm with >=sys-libs/gdbm-1.17 does not report lookup failures correctly
Summary: dev-libs/cyrus-sasl USE=gdbm with >=sys-libs/gdbm-1.17 does not report lookup...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal with 1 vote (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2019-02-03 16:23 UTC by Jonas Jelten
Modified: 2022-02-23 00:51 UTC (History)
3 users (show)

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

Attachments
copy the correct errno value to check against later (0001-db_gdbm-fix-gdbm_errno-overlay-from-gdbm_close.patch,1.05 KB, patch)
2019-02-03 16:44 UTC, Jonas Jelten 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jonas Jelten 2019-02-03 16:23:11 UTC 
gdbm_close of >=sys-libs/gdbm-1.17 also sets gdbm_errno. This overlays a value that is used for returning a lookup failure.

archlinux bug: https://bugs.archlinux.org/task/59873

upstream patch: https://github.com/cyrusimap/cyrus-sasl/pull/554
Comment 1 Jonas Jelten 2019-02-03 16:44:25 UTC 
Created attachment 563674 [details, diff]
copy the correct errno value to check against later

patch fixing the issue, also submitted upstream: https://github.com/cyrusimap/cyrus-sasl/pull/554
Comment 2 Holger Hoffstätte 2020-02-16 14:43:42 UTC 
Can confirm that the patch works and restores the ability to migrate cyrus-sasl from berkdb to gdbm (which otherwise fails with very misleading auth errors).
My system is now completely berkdb-free. Thank You!
Comment 3 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-02-23 00:51:29 UTC 
FWIW, this patch did actually get added at some point: https://gitweb.gentoo.org/repo/gentoo.git/commit/dev-libs/cyrus-sasl?id=0ea0e5c7d34102447316e0ec3e9a4aa82e4523bf