[Bug glob/28792] New: possible wrong behaviour with patterns with double [ with no closing ]

[Bug glob/28792] New: possible wrong behaviour with patterns with double [ with no closing ]

[calestyo at scientia dot org]

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

            Bug ID: 28792
           Summary: possible wrong behaviour with patterns with double [
                    with no closing ]
           Product: glibc
           Version: 2.33
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: glob
          Assignee: unassigned at sourceware dot org
          Reporter: calestyo at scientia dot org
  Target Milestone: ---

Hey.

Forwarding this from:

https://lore.kernel.org/dash/YeZbt7nhvODBSL0I@gondor.apana.org.au/T/#mb9438ab3009b9e38a2e22fdd22e99ddd74078b93

where it was suggested that this may be a bug in fnmatch().



Assuming a pattern of:
[.*^\]
my understanding was that this would actually mean:
- the literal string [. followed by
- the pattern notation special character * (i.e. any string) followed by
- the literal string ^]

Because ] is escaped, it's to be taken literally and in a pattern a [
without corresponding ] is to be taken literally as well.
(see POSIX,
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_13
)


That seems to work in dash (which according to its maintainer uses fnmatch()
for the matching):
$ case '[.^]' in ([.*^\]) echo match;; (*) echo no match;; esac
match
$ case '[.x^]' in ([.*^\]) echo match;; (*) echo no match;; esac
match
$ case '[.xx^]' in ([.*^\]) echo match;; (*) echo no match;; esac
match


However, adding another [ to the pattern:
[.*^[\]
which should be:
- the literal string [. followed by
- the pattern notation special character * (i.e. any string) followed by
- the literal string ^[]

No longer matches:
$ case '[.^[]' in ([.*^[\]) echo match;; (*) echo no match;; esac
no match
$ case '[.x^[]' in ([.*^[\]) echo match;; (*) echo no match;; esac
no match
$ case '[.xx^[]' in ([.*^[\]) echo match;; (*) echo no match;; esac
no match


Whereas, AFAIU POSIX, it should.

This works, btw. in bash (no idea whether that uses fnmatch() too), but neither
in busybox sh, nor klibc sh.


Thanks,
Chris.

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

[Bug glob/28792] possible wrong behaviour with patterns with double [ with no closing ]

[schwab@linux-m68k.org]

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

Andreas Schwab <schwab@linux-m68k.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|UNCONFIRMED                 |RESOLVED
   Target Milestone|---                         |2.35

--- Comment #1 from Andreas Schwab <schwab@linux-m68k.org> ---
Fixed in 2.35.

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

[Bug glob/28792] possible wrong behaviour with patterns with double [ with no closing ]

[calestyo at scientia dot org]

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

--- Comment #2 from Christoph Anton Mitterer <calestyo at scientia dot org> ---
Thanks :-)

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