Skip to content

Commit fe3a635

Browse files
authored
Merge pull request SAML-Toolkits#222 from thompsa/multicert
If the certificate arrays happen to be sparsely indexed this can blow…
2 parents b41f731 + dd46a0b commit fe3a635

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

lib/Saml2/Settings.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -941,13 +941,13 @@ public function formatIdPCertMulti()
941941
{
942942
if (isset($this->_idp['x509certMulti'])) {
943943
if (isset($this->_idp['x509certMulti']['signing'])) {
944-
for ($i=0; $i < count($this->_idp['x509certMulti']['signing']); $i++) {
945-
$this->_idp['x509certMulti']['signing'][$i] = OneLogin_Saml2_Utils::formatCert($this->_idp['x509certMulti']['signing'][$i]);
944+
foreach($this->_idp['x509certMulti']['signing'] as $i => $cert) {
945+
$this->_idp['x509certMulti']['signing'][$i] = OneLogin_Saml2_Utils::formatCert($cert);
946946
}
947947
}
948948
if (isset($this->_idp['x509certMulti']['encryption'])) {
949-
for ($i=0; $i < count($this->_idp['x509certMulti']['encryption']); $i++) {
950-
$this->_idp['x509certMulti']['encryption'][$i] = OneLogin_Saml2_Utils::formatCert($this->_idp['x509certMulti']['encryption'][$i]);
949+
foreach($this->_idp['x509certMulti']['encryption'] as $i => $cert) {
950+
$this->_idp['x509certMulti']['encryption'][$i] = OneLogin_Saml2_Utils::formatCert($cert);
951951
}
952952
}
953953
}

0 commit comments

Comments
 (0)