diff options
author | des <des@FreeBSD.org> | 2003-06-18 09:44:09 +0000 |
---|---|---|
committer | des <des@FreeBSD.org> | 2003-06-18 09:44:09 +0000 |
commit | 49e0f275db6cafef33693cb9ac1fdd52cdd63a67 (patch) | |
tree | 7a0c87730c03e720760c03e894817455d157a988 /contrib/openpam/lib/pam_end.c | |
parent | f4ecd4746647669fad236aa0e46da57f7761a0da (diff) | |
download | FreeBSD-src-49e0f275db6cafef33693cb9ac1fdd52cdd63a67.zip FreeBSD-src-49e0f275db6cafef33693cb9ac1fdd52cdd63a67.tar.gz |
Avoid side effects in macro arguments (perforce change #33323)
Submitted by: Dmitry V. Levin <ldv@altlinux.org>
Diffstat (limited to 'contrib/openpam/lib/pam_end.c')
-rw-r--r-- | contrib/openpam/lib/pam_end.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/contrib/openpam/lib/pam_end.c b/contrib/openpam/lib/pam_end.c index e1762c6..0dd3ea7 100644 --- a/contrib/openpam/lib/pam_end.c +++ b/contrib/openpam/lib/pam_end.c @@ -31,7 +31,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. * - * $P4: //depot/projects/openpam/lib/pam_end.c#13 $ + * $P4: //depot/projects/openpam/lib/pam_end.c#14 $ */ #include <stdlib.h> @@ -68,8 +68,10 @@ pam_end(pam_handle_t *pamh, } /* clear environment */ - while (pamh->env_count) - FREE(pamh->env[--pamh->env_count]); + while (pamh->env_count) { + --pamh->env_count; + FREE(pamh->env[pamh->env_count]); + } FREE(pamh->env); /* clear chains */ |