[Bug c++/64926] New: Variable declared in if-statement-header is in wrong scope

[Bug c++/64926] New: Variable declared in if-statement-header is in wrong scope

[humbug at deeptown dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64926

            Bug ID: 64926
           Summary: Variable declared in if-statement-header is in wrong
                    scope
           Product: gcc
           Version: 4.8.2
            Status: UNCONFIRMED
          Severity: major
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: humbug at deeptown dot org

A friend of mine (Dmitry Kashitsyn) found a bug:

#include <cstdio>

int main()
{    
    if (int* x=NULL) // <<< declare 'x'
    {}
    else if(true)
    {
        std::printf("%d\n", *x); // <<< You may access 'x' inside this scope
    }
    return 0;
}

[Bug c++/64926] Variable declared in if-statement-header is in wrong scope

[humbug at deeptown dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64926

Roman Proskuryakov <humbug at deeptown dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |accepts-invalid, diagnostic
           Severity|major                       |critical

[Bug c++/64926] Variable declared in if-statement-header is in wrong scope

[humbug at deeptown dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64926

--- Comment #1 from Roman Proskuryakov <humbug at deeptown dot org> ---
$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.8/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.8.2-19ubuntu1'
--with-bugurl=file:///usr/share/doc/gcc-4.8/README.Bugs
--enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr
--program-suffix=-4.8 --enable-shared --enable-linker-build-id
--libexecdir=/usr/lib --without-included-gettext --enable-threads=posix
--with-gxx-include-dir=/usr/include/c++/4.8 --libdir=/usr/lib --enable-nls
--with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug
--enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-libmudflap
--enable-plugin --with-system-zlib --disable-browser-plugin
--enable-java-awt=gtk --enable-gtk-cairo
--with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.8-amd64/jre --enable-java-home
--with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.8-amd64
--with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.8-amd64
--with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar
--enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686
--with-abi=m64 --with-multilib-list=m32,m64,mx32 --with-tune=generic
--enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu
--target=x86_64-linux-gnu
Thread model: posix
gcc version 4.8.2 (Ubuntu 4.8.2-19ubuntu1)

[Bug c++/64926] Variable declared in if-statement-header is in wrong scope

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64926

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |INVALID

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
No GCC is correct, the scope of the variable x is for both the true part and
the false part of the if statement.  You can't declare another variable with
the name of x in either part though but in your case we have a single statement
of if(true) {..} which means you can declare another variable of the same name
in the {} part as it is an inner-scope.

[Bug c++/64926] Variable declared in if-statement-header is in wrong scope

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64926

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|critical                    |normal