Home page logo
/

wireshark logo Wireshark mailing list archives

Re: rev 50749: /trunk/ /trunk/: CMakeLists.txt
From: Alexis La Goutte <alexis.lagoutte () gmail com>
Date: Wed, 7 Aug 2013 18:27:48 +0200

On Tue, Jul 23, 2013 at 8:57 PM, Guy Harris <guy () alum mit edu> wrote:


On Jul 22, 2013, at 7:00 AM, Alexis La Goutte <alexis.lagoutte () gmail com>
wrote:

I try also in my Ubunux box (12.04) with clang 3.0 ( Ubuntu clang
version 3.0-6ubuntu3 (tags/RELEASE_30/final) (based on LLVM 3.0) Target:
x86_64-pc-linux-gnu Thread model: posix )

and i have the following error when i try to build with clang :

[  4%] Building C object wiretap/CMakeFiles/wiretap.dir/file_wrappers.c.o
/home/alagoutte/wireshark2/wiretap/file_wrappers.c:818:13: error:
implicit conversion loses integer precision:
      '__blksize_t' (aka 'long') to 'int' [-Werror,-Wshorten-64-to-32]
                want = st.st_blksize;
                     ~ ~~~^~~~~~~~~~

I guess Linux is prepared for the world when files should be read in
chunks bigger than 2GB.

I've checked in a change to check whether st.st_blksize <= G_MAXINT and:

        if it is, cast st.st_blksize to int before assigning it;

        if it isn't, just leave "want" at the default value.

Try it now.

Have also for all file the following warning...
warning: unknown warning option '-Wpragmas' [-Wunknown-warning-option]

This change:


http://anonsvn.wireshark.org/viewvc/trunk/CMakeLists.txt?r1=50760&r2=50759&pathrev=50760

*should* have made CMake use -Werror=unknown-warning-option when testing
whether -W options are supported if the compiler supports
-Werror=unknown-warning-option, so it *should* leave out all -W options
that clang doesn't support.

If that's not happening, that needs to be fixed.


Hi,

I have always some errors when i build with clang 3.0 (with cmake or
autotools):

dumpcap.c:3981:88: error: implicit conversion loses integer precision:
'const __suseconds_t'
      (aka 'const long') to 'guint32' (aka 'unsigned int')
[-Werror,-Wshorten-64-to-32]

phdr->ts.tv_sec, phdr->ts.tv_usec,

~~~~~~~~~^~~~~~~
dumpcap.c:3989:73: error: implicit conversion loses integer precision:
'const __suseconds_t'
      (aka 'const long') to 'guint32' (aka 'unsigned int')
[-Werror,-Wshorten-64-to-32]
                                              phdr->ts.tv_sec,
phdr->ts.tv_usec,

~~~~~~~~~^~~~~~~
wiretap/lanalyzer.c:643:24: error: implicit conversion loses integer
precision: 'const __suseconds_t'
      (aka 'const long') to 'gint32' (aka 'int')
[-Werror,-Wshorten-64-to-32]
      gint32 usec = a->tv_usec;
             ~~~~   ~~~^~~~~~~

Fixed with add (gint32) cast before tv_usec :

-      gint32 usec = a->tv_usec;
+      gint32 usec = (gint32)a->tv_usec;

I have also a another issue :

dumpcap.c:4273:20: error: array index of '3' indexes past the end of an
array
      (that contains 3 elements) [-Werror,-Warray-bounds]
        if (strcmp("-Z", argv[i]) == 0) {
            ~~~~~~~^~~~~~~~~~~~~~
/usr/include/x86_64-linux-gnu/bits/string2.h:812:21: note: expanded from:
            : __strcmp_cg (s1, s2,
__s1_len))                                 \
                           ^
/usr/include/x86_64-linux-gnu/bits/string2.h:885:27: note: expanded from:
                                          (__const char *)
(s1))[3]          \
                                                             ^    ~
But now yet found a fix...
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe

  By Date           By Thread  

Current thread:
  • Re: rev 50749: /trunk/ /trunk/: CMakeLists.txt Alexis La Goutte (Aug 07)
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]
AlienVault