Projet

Général

Profil

Development #52961

lasso_saml2_assertion_set_basic_conditions sets OneTimeUse twice

Ajouté par Maxime Besson il y a environ 3 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Benjamin Dauvergne
Catégorie:
-
Version cible:
-
Début:
12 avril 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Trying to set a "one time use" condition on a SAML assertion results in the <OneTimeUse> tag being added twice in the XML assertion.

This is explicitely forbidden by saml-core:

A SAML authority MUST NOT include more than one <OneTimeUse> element within a
<Conditions>element of an assertion

Looking at the code hints at a mistake in lasso_saml2_assertion_set_one_time_use:

    # Added here
    lasso_list_add_new_gobject (saml2_conditions->OneTimeUse, lasso_saml2_one_time_use_new());
    if (one_time_use) {
        # re-added here
        lasso_list_add_new_gobject(saml2_conditions->OneTimeUse,
                lasso_saml2_one_time_use_new());

I have attached a test case that reproduces the issue on Debian 10.9/Lasso 2.6.1

Thanks for your consideration

Fichiers

lasso-one-time-use-appears-twice.c (324 octets) lasso-one-time-use-appears-twice.c Maxime Besson, 12 avril 2021 14:31
0001-Prevent-multiple-OneTimeUse-elements-52961.patch (1,41 ko) 0001-Prevent-multiple-OneTimeUse-elements-52961.patch Benjamin Dauvergne, 24 juin 2021 02:22

Révisions associées

Révision 663c094e (diff)
Ajouté par Benjamin Dauvergne il y a presque 3 ans

Prevent multiple OneTimeUse elements (#52961)

"A SAML authority MUST NOT include more than one <OneTimeUse> element within a
<Conditions>element of an assertion"

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

  • Assigné à mis à Benjamin Dauvergne
#2

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

#4

Mis à jour par Thomas Noël il y a presque 3 ans

  • Statut changé de Solution proposée à Solution validée
#6

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4a9d6f6ca3f90e33aea16ae2b35023985e1207a5
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Jun 24 02:21:14 2021 +0200

    Prevent multiple OneTimeUse elements (#52961)

    "A SAML authority MUST NOT include more than one <OneTimeUse> element within a
    <Conditions>element of an assertion"
#7

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF