Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

aja: Fix format-security warning which lead to a build error on some Linux distributions #5766

Merged
merged 1 commit into from
Jan 11, 2022

Conversation

tytan652
Copy link
Collaborator

@tytan652 tytan652 commented Jan 9, 2022

Description

Issue introduced by f09137a.

On some Linux distributions -Werror=format-security is set when building package keeping OBS Studio from being built with AJA plugins.

Motivation and Context

Allow OBS Studio to be build with -Werror=format-security set. Which is the case on ArchLinux build system.

How Has This Been Tested?

Build OBS Studio with makepkg.

  • On master it would emit an error.
  • On this branch it builds.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

On some Linux distributions -Werror=format-security is set when
building package keeping OBS Studio from being built with AJA
plugins.
@tytan652 tytan652 changed the title aja: Fix format-security warning which lead to a build error on some Linux distribution aja: Fix format-security warning which lead to a build error on some Linux distributions Jan 9, 2022
@WizardCM WizardCM added Bug Fix Non-breaking change which fixes an issue Linux Affects Linux labels Jan 9, 2022
@Gol-D-Ace
Copy link
Member

Also needed under Ubuntu from what I have seen by trying to build the PPA.
PR appears to fix the mentioned issue.

@WizardCM WizardCM merged commit 9d02ca0 into obsproject:master Jan 11, 2022
@tytan652 tytan652 deleted the fix_format-security branch January 11, 2022 09:50
@RytoEX RytoEX added this to the OBS Studio 27.2 milestone Jan 12, 2022
@RytoEX RytoEX added this to PRs Pending Review in OBS Studio 27.2 via automation Jan 12, 2022
@RytoEX RytoEX moved this from PRs Pending Review to Completed in OBS Studio 27.2 Jan 12, 2022
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 18, 2022
Some Linux package build system enables it.

obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 18, 2022
Some Linux package build system enables it.

obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 18, 2022
Some Linux package build system enables it.

obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 18, 2022
Deprecation warnings are not treated as errors.

-Wformat-securityis enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 18, 2022
Deprecation warnings are not treated as errors.

-Wformat-securityis enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 19, 2022
Deprecation warnings are not treated as errors.

-Wformat-securityis enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 19, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 19, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 19, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 19, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 21, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 21, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 21, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 21, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 22, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 23, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 24, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 24, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jul 24, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Aug 27, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Sep 1, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Sep 19, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Oct 2, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Oct 18, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Oct 21, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Oct 25, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Oct 25, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Oct 26, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Oct 31, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 5, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wno-error=deprecated-declarations is added on Linux/FreeBSD for release
builds to help packager.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 5, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 5, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 11, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 11, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 12, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 18, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 18, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 18, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 20, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Nov 22, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Dec 10, 2022
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jan 7, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jan 13, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jan 15, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is keep as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jan 18, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is kept as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jan 18, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is kept as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
tytan652 added a commit to tytan652/obs-studio that referenced this pull request Jan 19, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is kept as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
RytoEX pushed a commit to tytan652/obs-studio that referenced this pull request Jan 19, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is kept as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
RytoEX pushed a commit that referenced this pull request Jan 19, 2023
-Wformat-security is enabled, some Linux packages build system have it
enabled.
#5766

-Wunused-parameter is enabled too.

Also ensure that null conversion (C++ only for GCC) is enabled because its
enablement by default can depend on the CMake generator.

-Wswitch is enabled but it is kept as a warning. It was already enabled
with Clang.

If Clang has -Wshorten-64-to-32 enabled, keep it as warnings.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fix Non-breaking change which fixes an issue Linux Affects Linux
Projects
OBS Studio 27.2
  
Completed
Development

Successfully merging this pull request may close these issues.

None yet

4 participants