diff options
author | Damien Miller <djm@mindrot.org> | 2007-09-17 16:09:15 +1000 |
---|---|---|
committer | Damien Miller <djm@mindrot.org> | 2007-09-17 16:09:15 +1000 |
commit | 14b017d6f2277c83856489054e9c6c1428095bbb (patch) | |
tree | cc10c13ca8483de9ec16695a35de2355c90c29ef /sshlogin.c | |
parent | 9c89c837ccfeafb09f3d6fb463942fc919f3dc29 (diff) | |
download | openssh-git-14b017d6f2277c83856489054e9c6c1428095bbb.tar.gz |
- gilles@cvs.openbsd.org 2007/09/11 15:47:17
[session.c ssh-keygen.c sshlogin.c]
use strcspn to properly overwrite '\n' in fgets returned buffer
ok pyr@, ray@, millert@, moritz@, chl@
Diffstat (limited to 'sshlogin.c')
-rw-r--r-- | sshlogin.c | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -1,4 +1,4 @@ -/* $OpenBSD: sshlogin.c,v 1.25 2006/08/03 03:34:42 deraadt Exp $ */ +/* $OpenBSD: sshlogin.c,v 1.26 2007/09/11 15:47:17 gilles Exp $ */ /* * Author: Tatu Ylonen <ylo@cs.hut.fi> * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland @@ -98,8 +98,7 @@ store_lastlog_message(const char *user, uid_t uid) if (last_login_time != 0) { time_string = ctime(&last_login_time); - if (strchr(time_string, '\n')) - *strchr(time_string, '\n') = '\0'; + time_string[strcspn(time_string, "\n")] = '\0'; if (strcmp(hostname, "") == 0) snprintf(buf, sizeof(buf), "Last login: %s\r\n", time_string); |