view src/chomp.c @ 753:b58fd7980358

Fix theoretically possible memory corruption by chomp() A quick survey of callers of chomp() suggests this never could occur in practice
author Ben Schmidt
date Wed, 06 Oct 2010 23:26:26 +1100
parents 9935bcc6fc36
children
line wrap: on
line source

/* Copyright (C) 2002, 2003 Mads Martin Joergensen <mmj at mmj.dk>
 *
 * $Id$
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to
 * deal in the Software without restriction, including without limitation the
 * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or
 * sell copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in
 * all copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
 * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
 * IN THE SOFTWARE.
 */

#include <string.h>
#include "chomp.h"

char *chomp(char *str)
{
	size_t i;

	if(str == NULL)
		return NULL;

	if(*str == '\0' || *str == '\n' || *str == '\r') {
		*str = '\0';
		return str;
	}

	i = strlen(str) - 1;

	while((str[i] == '\n') || (str[i] == '\r')) {
		str[i] = 0;
		i--;
	}

	return str;
}