diff options
author | Simo Sorce <simo@redhat.com> | 2016-01-14 10:24:17 -0500 |
---|---|---|
committer | Simo Sorce <simo@redhat.com> | 2016-01-14 10:24:17 -0500 |
commit | 7c0d8634541da5804f711a0e777b3ab03e897911 (patch) | |
tree | afd62dd58d30cb7266f14e48fa127da18caec359 | |
parent | 3652da99739cc95c8278ba726e7446ec1e9c9e0f (diff) | |
download | mod_auth_gssapi-7c0d8634541da5804f711a0e777b3ab03e897911.tar.gz mod_auth_gssapi-7c0d8634541da5804f711a0e777b3ab03e897911.tar.xz mod_auth_gssapi-7c0d8634541da5804f711a0e777b3ab03e897911.zip |
Fix potential loop when requesting attribute data.
If this function fail we are better off abandoning the whole quest, continueing
here may end us up in an infinite loop where the fucntion keeps failing w/o
changing attr.more
Thanks to Alejandro Perez for finding this flaw.
Signed-off-by: Simo Sorce <simo@redhat.com>
-rw-r--r-- | src/environ.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/environ.c b/src/environ.c index 1ea72e0..f9bbf30 100644 --- a/src/environ.c +++ b/src/environ.c @@ -214,7 +214,7 @@ void mag_get_name_attributes(request_rec *req, struct mag_config *cfg, attr.value = empty_buffer; attr.display_value = empty_buffer; - if (!mag_get_name_attr(req, name, &attr)) continue; + if (!mag_get_name_attr(req, name, &attr)) break; if (cfg->name_attributes->output_json) { mag_add_json_name_attr(req, i == 0, &attr, &json); |