Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130221005634.GM20323@brightrain.aerifal.cx>
Date: Wed, 20 Feb 2013 19:56:35 -0500
From: Rich Felker <dalias@...ifal.cx>
To: musl@...ts.openwall.com
Subject: Re: Fix for fields in utmp

On Wed, Feb 20, 2013 at 07:49:37PM +0100, Szabolcs Nagy wrote:
> * Chris Spiegel <cspiegel@...il.com> [2013-02-20 10:23:09 -0800]:
> > The utmp.h header defines a few macros to access __ut_exit in the utmpx
> > struct; however, the underscores were removed in utmpx, so the macros
> > now point to non-existent struct members.  Attached is a simple patch
> > which removes them.
> 
> note that the current code is not compatible with glibc
> 
> http://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/gnu/bits/utmpx.h
> vs
> http://git.musl-libc.org/cgit/musl/tree/include/utmpx.h
> 
> it seems the __ prefix depends on _GNU_SOURCE

The versions with the __ prefix are never intended to be accessed by
name; they're just there to get the padding right. If some programs do
use them, however, we might need to provide them... BTW this is all
stub code; musl does not use utmp.

> and the 64bit abi is different on musl
> (see ut_session and ut_tv)

That should be fixed. Since it's all stubs, I don't think ABI is a big
issue here.

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.