[Bug gas/23453] segmentation fault in config/tc-i386.c, line 3989

[cvs-commit at gcc dot gnu.org]

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

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot 
gnu.org> ---
The master branch has been updated by H.J. Lu <address@hidden>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=1286ab788188bb8f0b0da2a3def4641fb3817f10

commit 1286ab788188bb8f0b0da2a3def4641fb3817f10
Author: H.J. Lu <address@hidden>
Date:   Fri Jul 27 06:35:09 2018 -0700

    x86: Check for more than 2 memory references

    For

    movsd (%esi), %ss:(%edi), %ss:(%eax)

    we got

    address@hidden tmp]$ as -o x.o x.s
    x.s: Assembler messages:
    x.s:1: Error: too many memory references for `movsd'
    munmap_chunk(): invalid pointer
    x.s:1: Internal error (Aborted).
    Please report this bug.
    address@hidden tmp]$

    struct _i386_insn has

    const seg_entry *seg[2];

    3 memory references will overflow the seg array.  We should issue an
    error if there are more than 2 memory references.

        PR gas/23453
        * config/tc-i386.c (parse_operands): Check for more than 2
        memory references.
        * testsuite/gas/i386/inval.s: Add a movsd test with 3 memory
        references.
        * testsuite/gas/i386/x86-64-inval.s: Likewise.
        * testsuite/gas/i386/inval.l: Updated.
        * testsuite/gas/i386/x86-64-inval.l: Likewise.

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