aft: Unify handling of hash and path duplicates.
authorAndre Noll <maan@tuebingen.mpg.de>
Mon, 1 Jun 2015 07:36:28 +0000 (09:36 +0200)
committerAndre Noll <maan@tuebingen.mpg.de>
Wed, 12 Aug 2015 21:23:48 +0000 (23:23 +0200)
commit99708bef7ad12ccb9399186f6055004d11bcf3db
tree27fba243ef8e5ce2164b46919e5dec3c9f2a4f0d
parentb4418a6993c2c146c6da048d027254c8dd56799e
aft: Unify handling of hash and path duplicates.

These are symmetric but were not treated as such. Specifically:

* We ignored fatal errors for finding hash sisters while path
brothers were handled strictly.

* For hash sisters, we had a helper which implicitly special
cased the non-fatal E_OSL_RB_KEY_NOT_FOUND case while the
same check was open-coded for path brothers.

This adds the find_path_brother() helper which is analogous to
find_hash_sister() and fixes up the latter function to perform strict
error checking rather than relying on the implementation detail that
osl_get_row() sets the result pointer to NULL if no entry was found.
aft.c