public inbox for libabigail@sourceware.org
 help / color / mirror / Atom feed
From: "sam at gentoo dot org" <sourceware-bugzilla@sourceware.org>
To: libabigail@sourceware.org
Subject: [Bug default/30503] New: Failure to compare libsdl-1.2.60 and libsdl-1.2.64
Date: Thu, 01 Jun 2023 00:33:53 +0000	[thread overview]
Message-ID: <bug-30503-9487@http.sourceware.org/bugzilla/> (raw)

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

            Bug ID: 30503
           Summary: Failure to compare libsdl-1.2.60 and libsdl-1.2.64
           Product: libabigail
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: default
          Assignee: dodji at redhat dot com
          Reporter: sam at gentoo dot org
                CC: libabigail at sourceware dot org
  Target Milestone: ---

As discussed on #libabigail a few days ago.

```
$ abidiff --version
abidiff: 2.3.0

$ abidiff 1.2.60/lib64/libSDL-1.2.so.1.2.60 1.2.64/lib64/libSDL-1.2.so.1.2.64
Functions changes summary: 0 Removed, 1 Changed (1 filtered out), 0 Added
functions
Variables changes summary: 0 Removed, 0 Changed, 0 Added variable

1 function with some indirect sub-type change:

  [C] 'function int SDL_BuildAudioCVT(SDL12_AudioCVT*, Uint16, Uint8, int,
Uint16, Uint8, int)' at SDL12_compat.c:9383:1 has some indirect sub-type
changes:
    parameter 1 of type 'SDL12_AudioCVT*' has sub-type changes:
      in pointed to type 'typedef SDL12_AudioCVT' at SDL12_compat.c:920:1:
        underlying type 'struct SDL12_AudioCVT' at SDL12_compat.c:880:1
changed:
          type size hasn't changed
          11 data member changes:
            'int needed' has *some* difference - please report as a bug
            'Uint16 src_format' has *some* difference - please report as a bug
            'Uint16 dst_format' has *some* difference - please report as a bug
            'double rate_incr' has *some* difference - please report as a bug
            'Uint8* buf' has *some* difference - please report as a bug
            'int len' has *some* difference - please report as a bug
            'int len_cvt' has *some* difference - please report as a bug
            'int len_mult' has *some* difference - please report as a bug
            'double len_ratio' has *some* difference - please report as a bug
            type of 'void (SDL_AudioCVT*, typedef Uint16)* filters[10]'
changed:
              array element type 'void (SDL_AudioCVT*, typedef Uint16)*'
changed:
                entity changed from 'void (SDL_AudioCVT*, typedef Uint16)*' to
compatible type 'typedef SDL12_AudioCVTFilter' at SDL12_compat.c:903:1
                  in pointed to type 'function type void (SDL_AudioCVT*,
typedef SDL_AudioFormat)':
                    parameter 1 of type 'SDL_AudioCVT*' changed:
                      in pointed to type 'struct SDL_AudioCVT' at
SDL12_compat.c:907:1:
                        type name changed from 'SDL_AudioCVT' to
'SDL12_AudioCVT'
                        type size changed from 1024 to 1088 (in bits)
                        3 data member changes (8 filtered):
                          'double len_ratio' offset changed from 288 to 320 (in
bits) (by +32 bits)
                          name of 'SDL_AudioCVT::filters' changed to
'SDL12_AudioCVT::filters' at SDL12_compat.c:918:1, offset changed from 352 to
384 (in bits) (by +32 bits)
                          'int filter_index' offset changed from 992 to 1024
(in bits) (by +32 bits)
              type name changed from 'void (SDL_AudioCVT*, typedef
Uint16)*[10]' to 'SDL12_AudioCVTFilter[10]'
              type size hasn't changed
            'int filter_index' has *some* difference - please report as a bug
```

I've attached a tarball with the relevant files. I included the 32-bit versions
(in lib/) for completeness (because I'd already copied before checking, oops..)
but the bug only happens w/ the 64-bit versions (in lib64/):
```
$ tree libsdl-libabigail/
libsdl-libabigail/
├── 1.2.60
│   ├── lib
│   │   ├── libSDL-1.2.so.1.2.60
│   │   └── libSDL-1.2.so.1.2.60.debug
│   └── lib64
│       ├── libSDL-1.2.so.1.2.60
│       └── libSDL-1.2.so.1.2.60.debug
└── 1.2.64
    ├── lib
    │   ├── libSDL-1.2.so.1.2.64
    │   └── libSDL-1.2.so.1.2.64.debug
    └── lib64
        ├── libSDL-1.2.so.1.2.64
        └── libSDL-1.2.so.1.2.64.debug

7 directories, 8 files
```

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

             reply	other threads:[~2023-06-01  0:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-01  0:33 sam at gentoo dot org [this message]
2023-06-01  0:34 ` [Bug default/30503] " sam at gentoo dot org
2023-06-05 16:57 ` dodji at redhat dot com
2023-06-07  9:57 ` [Bug default/30503] Failure to compare a member of a non-anonymous struct against a member of a typedef-named anonymous struct dodji at redhat dot com
2023-06-07 12:15 ` dodji at redhat dot com
2023-06-07 12:23 ` sam at gentoo dot org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-30503-9487@http.sourceware.org/bugzilla/ \
    --to=sourceware-bugzilla@sourceware.org \
    --cc=libabigail@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).