Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 828082 - media-video/rtmpdump-2.4_p20190330[-ssl] - rtmp.c: error: INT_MAX undeclared (first use in this function)
Summary: media-video/rtmpdump-2.4_p20190330[-ssl] - rtmp.c: error: INT_MAX undeclared ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Craig Andrews
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-12-03 15:39 UTC by Toralf Förster
Modified: 2022-05-12 20:42 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge-info.txt (emerge-info.txt,15.39 KB, text/plain)
2021-12-03 15:39 UTC, Toralf Förster
Details
emerge-history.txt (emerge-history.txt,196.94 KB, text/plain)
2021-12-03 15:39 UTC, Toralf Förster
Details
environment (environment,88.55 KB, text/plain)
2021-12-03 15:39 UTC, Toralf Förster
Details
etc.portage.tar.bz2 (etc.portage.tar.bz2,65.87 KB, application/x-bzip)
2021-12-03 15:39 UTC, Toralf Förster
Details
media-video:rtmpdump-2.4_p20190330:20211203-001517.log (media-video:rtmpdump-2.4_p20190330:20211203-001517.log,6.06 KB, text/plain)
2021-12-03 15:39 UTC, Toralf Förster
Details
temp.tar.bz2 (temp.tar.bz2,22.45 KB, application/x-bzip)
2021-12-03 15:39 UTC, Toralf Förster
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Toralf Förster gentoo-dev 2021-12-03 15:39:37 UTC
rtmp.c:3555:7: warning: variable ‘didAlloc’ set but not used [-Wunused-but-set-variable]
 3555 |   int didAlloc = FALSE;
      |       ^~~~~~~~
rtmp.c: In function ‘HTTP_read’:
rtmp.c:4459:26: error: ‘INT_MAX’ undeclared (first use in this function)
 4459 |   if (hlen < 1 || hlen > INT_MAX)
      |                          ^~~~~~~
rtmp.c:34:1: note: ‘INT_MAX’ is defined in header ‘<limits.h>’; did you forget to ‘#include <limits.h>’?
   33 | #include "log.h"

  -------------------------------------------------------------------

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 17.1_no_multilib-j4-20211201-080026

  -------------------------------------------------------------------

gcc-config -l:
 [1] x86_64-pc-linux-gnu-11.2.1 *
clang version 13.0.0
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/lib/llvm/13/bin
/usr/lib/llvm/13
13.0.0
Python 3.9.9
Available Ruby profiles:
  [1]   ruby26 (with Rubygems)
  [2]   ruby27 (with Rubygems)
  [3]   ruby30 (with Rubygems) *
Available Rust versions:
  [1]   rust-1.56.1 *
The Glorious Glasgow Haskell Compilation System, version 8.10.4
php cli:
  [1]   php7.3
  [2]   php7.4
  [3]   php8.1 *

  HEAD of ::gentoo
commit 44a215e68a0ce7adfd4aaa3658d624ab31a4ab62
Author: Repository mirror & CI <repomirrorci@gentoo.org>
Date:   Thu Dec 2 23:06:44 2021 +0000

    2021-12-02 23:06:43 UTC

emerge -qpvO media-video/rtmpdump
[ebuild  N    ] media-video/rtmpdump-2.4_p20190330  USE="tools -gnutls -ssl -static-libs"
Comment 1 Toralf Förster gentoo-dev 2021-12-03 15:39:38 UTC
Created attachment 757271 [details]
emerge-info.txt
Comment 2 Toralf Förster gentoo-dev 2021-12-03 15:39:40 UTC
Created attachment 757272 [details]
emerge-history.txt
Comment 3 Toralf Förster gentoo-dev 2021-12-03 15:39:41 UTC
Created attachment 757273 [details]
environment
Comment 4 Toralf Förster gentoo-dev 2021-12-03 15:39:42 UTC
Created attachment 757274 [details]
etc.portage.tar.bz2
Comment 5 Toralf Förster gentoo-dev 2021-12-03 15:39:43 UTC
Created attachment 757275 [details]
media-video:rtmpdump-2.4_p20190330:20211203-001517.log
Comment 6 Toralf Förster gentoo-dev 2021-12-03 15:39:45 UTC
Created attachment 757276 [details]
temp.tar.bz2
Comment 7 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-05-08 04:49:48 UTC
candrews, could you try a newer snapshot if one exists? cheers!
Comment 8 Craig Andrews gentoo-dev 2022-05-09 13:29:45 UTC
(In reply to Sam James from comment #7)
> candrews, could you try a newer snapshot if one exists? cheers!

There's only 1 newer commit than our current snapshot... rtmpdump isn't exactly active.

https://git.ffmpeg.org/rtmpdump.git

I checked Debian and Alpine, neither have a mention of this issue or a patch fixing issue.

If someone provides (or links to) a patch, I'm happy to apply it.
Comment 9 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-05-09 13:32:24 UTC
(In reply to Craig Andrews from comment #8)
> (In reply to Sam James from comment #7)
> > candrews, could you try a newer snapshot if one exists? cheers!
> 
> There's only 1 newer commit than our current snapshot... rtmpdump isn't
> exactly active.
> 
> https://git.ffmpeg.org/rtmpdump.git
> 
> I checked Debian and Alpine, neither have a mention of this issue or a patch
> fixing issue.
> 
> If someone provides (or links to) a patch, I'm happy to apply it.

Ah, sorry, I'd tried looking but I couldn't find anything (even the repo, but that's my bad).

Patch should be easy:
>rtmp.c:34:1: note: ‘INT_MAX’ is defined in header ‘<limits.h>’; did you forget to ‘#include <limits.h>’?

I'll look now though.
Comment 10 Mike Gilbert gentoo-dev 2022-05-12 20:29:16 UTC
I don't think this is actually related to openssl-3.0, so I'm removing it from the tracker.
Comment 11 Larry the Git Cow gentoo-dev 2022-05-12 20:42:58 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8bf393aded52c35ade125bc216310cd54e8f5b2a

commit 8bf393aded52c35ade125bc216310cd54e8f5b2a
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-05-12 20:42:22 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-05-12 20:42:51 +0000

    media-video/rtmpdump: fix build with GCC 11
    
    Think it's 11+ specific (obviously these things depend
    heavily on env. as it's to do with transitive includes), but
    only occurs with *just* USE=tools enabled.
    
    Closes: https://bugs.gentoo.org/828082
    Signed-off-by: Sam James <sam@gentoo.org>

 ...dump-2.4_p20210219-gcc-11-missing-include.patch | 23 ++++++++++++++++++++++
 media-video/rtmpdump/rtmpdump-2.4_p20210219.ebuild |  1 +
 2 files changed, 24 insertions(+)