Message ID | 53B90C35.6070501@gmail.com |
---|---|
State | New |
Headers | show |
06.07.2014 12:43, Chen Gang wrote: > getchar() is a standard c library function which may return with failure > (e.g. -1), so like another platforms, also need check it under WIN32. Applied to -trivial queue, with a slight modification: > --- a/qemu-img.c > +++ b/qemu-img.c > @@ -185,15 +185,21 @@ static int GCC_FMT_ATTR(2, 3) qprintf(bool quiet, const char *fmt, ...) > static int read_password(char *buf, int buf_size) > { > int c, i; > + > printf("Password: "); > fflush(stdout); > i = 0; > for(;;) { > c = getchar(); > - if (c == '\n') > + if (c < 0) { > + buf[i] = '\0'; > + return -1; > + } else if (c == '\n') { > break; > - if (i < (buf_size - 1)) > + } > + if (i < (buf_size - 1)) { I've added an 'else' there and merged with the previous line, to match with the above code which is being added. > buf[i++] = c; > + } > } > buf[i] = '\0'; > return 0; Thanks, /mjt
On 08/02/2014 09:35 PM, Michael Tokarev wrote: > 06.07.2014 12:43, Chen Gang wrote: >> getchar() is a standard c library function which may return with failure >> (e.g. -1), so like another platforms, also need check it under WIN32. > > Applied to -trivial queue, with a slight modification: > Thanks, and I shall continue to make another patches for qemu within this month. >> --- a/qemu-img.c >> +++ b/qemu-img.c >> @@ -185,15 +185,21 @@ static int GCC_FMT_ATTR(2, 3) qprintf(bool quiet, const char *fmt, ...) >> static int read_password(char *buf, int buf_size) >> { >> int c, i; >> + >> printf("Password: "); >> fflush(stdout); >> i = 0; >> for(;;) { >> c = getchar(); >> - if (c == '\n') >> + if (c < 0) { >> + buf[i] = '\0'; >> + return -1; >> + } else if (c == '\n') { >> break; >> - if (i < (buf_size - 1)) >> + } >> + if (i < (buf_size - 1)) { > > I've added an 'else' there and merged with the previous line, > to match with the above code which is being added. > OK, thanks. >> buf[i++] = c; >> + } >> } >> buf[i] = '\0'; >> return 0; > > Thanks, > > /mjt > Thanks.
diff --git a/qemu-img.c b/qemu-img.c index c98896b..8423e8e 100644 --- a/qemu-img.c +++ b/qemu-img.c @@ -185,15 +185,21 @@ static int GCC_FMT_ATTR(2, 3) qprintf(bool quiet, const char *fmt, ...) static int read_password(char *buf, int buf_size) { int c, i; + printf("Password: "); fflush(stdout); i = 0; for(;;) { c = getchar(); - if (c == '\n') + if (c < 0) { + buf[i] = '\0'; + return -1; + } else if (c == '\n') { break; - if (i < (buf_size - 1)) + } + if (i < (buf_size - 1)) { buf[i++] = c; + } } buf[i] = '\0'; return 0;
getchar() is a standard c library function which may return with failure (e.g. -1), so like another platforms, also need check it under WIN32. And make the related code match current qemu code styles, too. Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> --- qemu-img.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)