Introduction

IMHO the rule #1 of any bug hunter (note I do not consider myself one of them since I do this really sporadically) is to have a good RSS feed list. In the course of the last years I built a pretty decent one and I try to follow other security experts trying to "steal" some useful tricks. There are many experts in different fields of the security panorama and too many to quote them here (maybe another post). But one of the leading expert (that I follow) on SAML is by far Ioannis Kakavas . Indeed he was able in the last years to find serious vulnerability in the SAML implementation of Microsoft and Github . Usually I am more an "OAuth guy" but since both, SAML and OAuth, are nothing else that grandchildren of Kerberos learning SAML has been in my todo list for long time. The Github incident gave me the final motivation to start learning it.

Learning SAML

SAML-idiot until begin of 2017 but then I decided to learn it a little bit. Of course I started giving a look the the specification, but the best way I learn things is by doing and hopefully breaking. So I downloaded SAML Raider (great stuff, it saves so much time, you can edit any assertion on the fly). As said I was a kind ofuntil begin of 2017 but then I decided to learn it a little bit. Of course I started giving a look the the specification, but the best way I learn things is by doing and hopefully breaking. So I downloaded this great Burp extension called(great stuff, it saves so much time, you can edit any assertion on the fly).

Then I tried to look if any of the service that routinely I use are SAML compliant. It turns out that many of them are. To name some:

Github (but I guess Ioannis already took all the bugs there). So ping next (I actually found this funny JS Github bug giving a look into it, but not pertinent here)

Hackerone, I gave a try here but nada, nisba, niente, nicht, niet

Slack, Bingo see next section (this is probably meant for Enterprise customers)

Slack SAML authentication bypass

Audience and AudienceRestriction . Quoting As said many of the service I use in my routine are SAML aware so I started to poke a bit them. The vulnerability I found is part of the class known as " confused deputy problem ". I already talked about it in one of my OAuth blog post (tl;dr this is also why you never want to use OAuth implicit grant flow as authentication mechanism) and is really simple. Basically SAML assertions, between others contains an element calledand. Quoting Ioannis





The Assertion also contains an AudienceRestriction element that defines that this Assertion is targeted for a specific Service Provider and cannot be used for any other Service Provider.

This means that if I present to a ServiceProvider A an assertion meant for ServiceProvider B, then the ServiceProvider A shoud reject it.

Well between all other things I tried this very really simple attack against a Slack's SAML endpoint /sso/saml and guess what? It worked :o !!

Disclosure timeline

Acknowledgement

Max Feldman

For more SAML trickery follow me on Twitter.

tl;dr I found a severe issue in the Slack 's SAML implementation that allowed me to bypass the authentication. This has now been solved by Slack.To be more concrete I used an old andyes the assertion was also expired!!) Github's Assertion I had saved somewhere in my archive that was signed for a subject different than mine (namely the username was not asanso aka me) and I presented to Slack. Slack happily accepted it and I was logged in Slack channel with the username of this old and expired Assertion that was never meant to be a Slack one!!! Wow this is scary.... Well well this look bad enough so I stopped quite immediately and open a ticket on Hackerone .......here the Slack security team was simply amazing... Thanks guysReported the issue via Hackerone.Slack confirmed the issue.Slack awarded a 3000$ bounty but still working with the affected customers in order to solve the vulnerability. Hence the ticket was kept open.Slack closed the issueI would like to thank the Slack security team in particularyou guys rock, really!!Well that's all folks.