[Bug build/26985] New: x86: sysdeps/x86/configure.ac needs quoting for CET support.

[Bug build/26985] New: x86: sysdeps/x86/configure.ac needs quoting for CET support.

[carlos at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=26985

            Bug ID: 26985
           Summary: x86: sysdeps/x86/configure.ac needs quoting for CET
                    support.
           Product: glibc
           Version: 2.33
            Status: NEW
          Severity: normal
          Priority: P2
         Component: build
          Assignee: unassigned at sourceware dot org
          Reporter: carlos at redhat dot com
                CC: carlos at redhat dot com
  Target Milestone: ---

Both of these checks need proper quoting and support for the case where the
variable is not defined.

  4 if test $enable_cet != no; then
  5   # Check if CET can be enabled.
  6   AC_CACHE_CHECK(whether CET can be enabled,
  7                  libc_cv_x86_cet_available, [dnl
  8 cat > conftest.c <<EOF
...
 23 if test $enable_cet != no; then
 24   # Check if assembler supports CET.
 25   AC_CACHE_CHECK(whether $AS supports CET,

We need quoting around "$enable_cet" and x-prefix to avoid breaking things if
the string contains an executable command by accident with dashes.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/26985] x86: sysdeps/x86/configure.ac needs quoting for CET support.

[carlos at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=26985

Carlos O'Donell <carlos at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hjl.tools at gmail dot com

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/26985] x86: sysdeps/x86/configure.ac needs quoting for CET support.

[fweimer at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=26985

Florian Weimer <fweimer at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fweimer at redhat dot com

--- Comment #1 from Florian Weimer <fweimer at redhat dot com> ---
I don't understand. $enable_cet is either yes or no, right?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/26985] x86: sysdeps/x86/configure.ac needs quoting for CET support.

[schwab@linux-m68k.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=26985

--- Comment #2 from Andreas Schwab <schwab@linux-m68k.org> ---
It will be whatever you pass to --enable-cet.

-- 
You are receiving this mail because:
You are on the CC list for the bug.