Marcus Meissner : regedit: fgetc() gets int, not char (Coverity).

Alexandre Julliard julliard at winehq.org
Wed Feb 23 13:01:54 CST 2011


Module: wine
Branch: master
Commit: 62f270e269517faf705228b3591ec6cdffa59830
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=62f270e269517faf705228b3591ec6cdffa59830

Author: Marcus Meissner <meissner at suse.de>
Date:   Wed Feb 23 17:20:43 2011 +0100

regedit: fgetc() gets int, not char (Coverity).

---

 programs/regedit/regproc.c |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/programs/regedit/regproc.c b/programs/regedit/regproc.c
index ff97970..f753947 100644
--- a/programs/regedit/regproc.c
+++ b/programs/regedit/regproc.c
@@ -664,8 +664,11 @@ static void processRegLinesA(FILE *in, char* first_chars)
             /* get a single line. note that `i' must be one past the last
              * meaningful character in `s' when this loop exits */
             for(i = 0; i < size_to_get-1; ++i){
-                s[i] = fgetc(in);
-                if(s[i] == EOF){
+                int xchar;
+
+                xchar = fgetc(in);
+                s[i] = xchar;
+                if(xchar == EOF){
                     if(ferror(in)){
                         perror("While reading input");
                         exit(IO_ERROR);




More information about the wine-cvs mailing list