Invalid invocation of vectorized copy2WithGain

Bug #1770771 reported by Uwe Klotz
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
RJ Skerry-Ryan

Bug Description

GCC 8.1.1 reveals aliasing of pointers that are passed as function arguments:

```
src/engine/enginemaster.cpp: In member function ‘void EngineMaster::process(int)’:

src/engine/enginemaster.cpp:554:43: warning: passing argument 1 to restrict-qualified parameter aliases with argument 2 [-Wrestrict]
                 SampleUtil::copy2WithGain(m_pMaster,
                                           ^~~~~~~~~
                     m_pMaster, 1.0,
                     ~~~~~~~~~
src/engine/enginemaster.cpp:585:43: warning: passing argument 1 to restrict-qualified parameter aliases with argument 2 [-Wrestrict]
                 SampleUtil::copy2WithGain(m_pMaster,
                                           ^~~~~~~~~
                     m_pMaster, 1.0,
                     ~~~~~~~~~
src/engine/enginemaster.cpp:664:43: warning: passing argument 1 to restrict-qualified parameter aliases with argument 2 [-Wrestrict]
                 SampleUtil::copy2WithGain(m_pSidechainMix,
                                           ^~~~~~~~~~~~~~~
                                           m_pSidechainMix, 1.0,
                                           ~~~~~~~~~~~~~~~
[CXX] src/engine/loopingcontrol.cpp
```

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Oh no.. that's a disaster waiting to happen.

Changed in mixxx:
importance: Undecided → High
status: New → Confirmed
milestone: none → 2.1.5
assignee: nobody → RJ Skerry-Ryan (rryan)
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Will cherry-pick to 2.2 once merged.

RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Confirmed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/9285

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.