[Bug libc/30266] New: Add test coverage for strdup() and strndup()

[Bug libc/30266] New: Add test coverage for strdup() and strndup()

[carlos at redhat dot com]

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

            Bug ID: 30266
           Summary: Add test coverage for strdup() and strndup()
           Product: glibc
           Version: 2.38
            Status: NEW
          Severity: normal
          Priority: P2
         Component: libc
          Assignee: unassigned at sourceware dot org
          Reporter: carlos at redhat dot com
                CC: drepper.fsp at gmail dot com
  Target Milestone: ---

The strdup() and strndup() function is largely implemented on top of malloc(),
but have close to no coverage testing for the individual functions and their
semantics (except for 1-3 small tests of simple behaviour).

We need to test these functions with various inputs, aligned, unaligned, and
under various conditions of null pointers etc.

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

[Bug libc/30266] Add test coverage for strdup() and strndup()

[carlos at redhat dot com]

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

--- Comment #1 from Carlos O'Donell <carlos at redhat dot com> ---
Should verify that they return malloc-aligned strings.

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

[Bug libc/30266] Add test coverage for strdup() and strndup()

[carlos at redhat dot com]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |good-first-bug

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

[Bug libc/30266] Add test coverage for strdup() and strndup()

[carlos at redhat dot com]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
   Target Milestone|---                         |2.38
             Status|NEW                         |RESOLVED

--- Comment #2 from Carlos O'Donell <carlos at redhat dot com> ---
Coverage was improved by Joe Simmons-Talbott.

commit eaaad78db41724e5a18a42becb238bfc4e683998
Author: Joe Simmons-Talbott <josimmon@redhat.com>
Date:   Fri Apr 21 09:24:23 2023 -0400

    string: Add tests for strdup (BZ #30266)

    Copy strcpy tests for strdup.  Covers some basic testcases with random
    strings.  Add a zero-length string testcase.

    Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

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