Ove Kaaven <ovehk(a)ping.uio.no> writes:
> Now that glibc 2.3 is available for those bleeding-edge folks on debian
> unstable, it seems that Wines compiled on 2.2 won't run on 2.3 systems.
> The glibc folks did warn us about this a while ago though, so here's a
> patch that implements their suggestion for making wine compatible with as
> many glibc versions as possible. Perhaps Alexandre wants to use his
> WINE_GET_SONAME stuff to detect the name of the libc at configure time
> instead of at runtime like I do, though.
Funny, I was working on the same thing right now... But do you really
need to fetch the real libc handle? It seems to me RTLD_NEXT should
be enough. Here's how I did it:
Index: scheduler/pthread.c
===================================================================
RCS file: /opt/cvs-commit/wine/scheduler/pthread.c,v
retrieving revision 1.26
diff -u -r1.26 pthread.c
--- scheduler/pthread.c 12 Sep 2002 22:07:06 -0000 1.26
+++ scheduler/pthread.c 20 Oct 2002 21:45:13 -0000
@@ -38,13 +38,6 @@
#include "thread.h"
#include "winternl.h"
-static int init_done;
-
-void PTHREAD_init_done(void)
-{
- init_done = 1;
-}
-
/* Currently this probably works only for glibc2,
* which checks for the presence of double-underscore-prepended
* pthread primitives, and use them if available.
@@ -62,28 +55,18 @@
asm(".globl " PSTR(alias) "\n" \
"\t.set " PSTR(alias) "," PSTR(orig))
-/* strong_alias does not work on external symbols (.o format limitation?),
- * so for those, we need to use the pogo stick */
-#if defined(__i386__) && !defined(__PIC__)
-/* FIXME: PIC */
-#define jump_alias(orig, alias) __ASM_GLOBAL_FUNC( alias, "jmp " PSTR(orig))
-#endif
+static int init_done;
-/* get necessary libc symbols */
-#if (__GLIBC__ == 2) && (__GLIBC_MINOR__ >= 1) && defined(HAVE___LIBC_FORK)
-#define LIBC_FORK __libc_fork
-#define PTHREAD_FORK __fork
-#define ALIAS_FORK
-#else
-#define LIBC_FORK __fork
-#define PTHREAD_FORK fork
-#endif
-extern pid_t LIBC_FORK(void);
+static pid_t (*libc_fork)(void);
+static int (*libc_sigaction)(int signum, const struct sigaction *act, struct sigaction *oldact);
+
+void PTHREAD_init_done(void)
+{
+ init_done = 1;
+ if (!libc_fork) libc_fork = dlsym( RTLD_NEXT, "fork" );
+ if (!libc_sigaction) libc_sigaction = dlsym( RTLD_NEXT, "sigaction" );
+}
-#define LIBC_SIGACTION __sigaction
-extern int LIBC_SIGACTION(int signum,
- const struct sigaction *act,
- struct sigaction *oldact);
/* NOTE: This is a truly extremely incredibly ugly hack!
* But it does seem to work... */
@@ -255,15 +238,20 @@
}
strong_alias(__pthread_atfork, pthread_atfork);
-pid_t PTHREAD_FORK(void)
+pid_t __fork(void)
{
pid_t pid;
int i;
+ if (!libc_fork)
+ {
+ libc_fork = dlsym( RTLD_NEXT, "fork" );
+ assert( libc_fork );
+ }
EnterCriticalSection( &atfork_section );
/* prepare handlers are called in reverse insertion order */
for (i = atfork_count - 1; i >= 0; i--) if (atfork_prepare[i]) atfork_prepare[i]();
- if (!(pid = LIBC_FORK()))
+ if (!(pid = libc_fork()))
{
InitializeCriticalSection( &atfork_section );
for (i = 0; i < atfork_count; i++) if (atfork_child[i]) atfork_child[i]();
@@ -275,9 +263,7 @@
}
return pid;
}
-#ifdef ALIAS_FORK
-strong_alias(PTHREAD_FORK, fork);
-#endif
+strong_alias(__fork, fork);
/***** MUTEXES *****/
@@ -658,13 +644,20 @@
/***** ANTI-OVERRIDES *****/
/* pthreads tries to override these, point them back to libc */
-#ifdef jump_alias
-jump_alias(LIBC_SIGACTION, sigaction);
-#else
int sigaction(int signum, const struct sigaction *act, struct sigaction *oldact)
{
- return LIBC_SIGACTION(signum, act, oldact);
+ if (!libc_sigaction)
+ {
+ libc_sigaction = dlsym( RTLD_NEXT, "sigaction" );
+ assert( libc_sigaction );
+ }
+ return libc_sigaction(signum, act, oldact);
+}
+
+#else /* __GLIBC__ */
+
+void PTHREAD_init_done(void)
+{
}
-#endif
#endif /* __GLIBC__ */
--
Alexandre Julliard
julliard(a)winehq.com