Home page logo
/

wireshark logo Wireshark mailing list archives

Re: [Wireshark-commits] rev 51169: / /trunk/epan/: app_mem_usage.c
From: Evan Huus <eapache () gmail com>
Date: Tue, 6 Aug 2013 14:18:38 -0400

On Tue, Aug 6, 2013 at 2:17 PM, Evan Huus <eapache () gmail com> wrote:
On Tue, Aug 6, 2013 at 2:14 PM, Jakub Zawadzki
<darkjames-ws () darkjames pl> wrote:
On Tue, Aug 06, 2013 at 10:59:42AM -0700, Guy Harris wrote:

On Aug 6, 2013, at 9:38 AM, darkjames () wireshark org wrote:

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=51169

User: darkjames
Date: 2013/08/06 09:38 AM

Log:
app_mem_usage fixes:

- Add missing feature test macro for pread()

Do all versions of the Linux kernel and of glibc have pread()?

Probably, from man:

The  pread()  and  pwrite() system calls were added to Linux in version 2.1.60
(...) C library support (including emulation using lseek(2) on older
kernels without the system calls) was added in glibc 2.1

glibc 2.1 was released about 1999

Hmm, we have conflicting man pages.
http://linux.die.net/man/2/pread

Or not quite. The question is why the feature test macro is for glibc
2.12 and 2008, when the function existed long before that...

If not, would an lseek() followed by a read() do just as well?

Sure.
___________________________________________________________________________
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
___________________________________________________________________________
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:
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]
AlienVault