diff options
author | Andreas Schwab <schwab@redhat.com> | 2010-04-13 07:13:00 -0700 |
---|---|---|
committer | Ulrich Drepper <drepper@redhat.com> | 2010-04-13 07:13:00 -0700 |
commit | ef634a94c7126bcd6ac02de495b598fb99ac544d (patch) | |
tree | 4eafe6664d2bdb1bc1e2582e8b642d026e61e49a /stdlib | |
parent | ea42a20caed5b343ff20a0d4622ae6c17b77161b (diff) | |
download | glibc-ef634a94c7126bcd6ac02de495b598fb99ac544d.zip glibc-ef634a94c7126bcd6ac02de495b598fb99ac544d.tar.gz glibc-ef634a94c7126bcd6ac02de495b598fb99ac544d.tar.bz2 |
Fix use of ucontext_t objects in tst-makecontext3
Objects of type ucontext_t cannot be copied, only getcontext can
properly initialize them. For example, on powerpc the structure
contains a pointer into itself, so makecontext modifies the original
object by side effect.
Diffstat (limited to 'stdlib')
-rw-r--r-- | stdlib/tst-makecontext3.c | 62 |
1 files changed, 33 insertions, 29 deletions
diff --git a/stdlib/tst-makecontext3.c b/stdlib/tst-makecontext3.c index f127c6a..a44169a 100644 --- a/stdlib/tst-makecontext3.c +++ b/stdlib/tst-makecontext3.c @@ -136,38 +136,42 @@ main (void) exit (1); } - ctx[1] = ctx[0]; + if (getcontext (&ctx[1]) != 0) + { + printf ("%s: getcontext: %m\n", __FUNCTION__); + exit (1); + } + ctx[1].uc_stack.ss_sp = st1; ctx[1].uc_stack.ss_size = sizeof st1; ctx[1].uc_link = &ctx[0]; - { - ucontext_t tempctx = ctx[1]; - makecontext (&ctx[1], (void (*) (void)) f1, 33, - 0x00000001 << flag, 0x00000004 << flag, - 0x00000012 << flag, 0x00000048 << flag, - 0x00000123 << flag, 0x0000048d << flag, - 0x00001234 << flag, 0x000048d1 << flag, - 0x00012345 << flag, 0x00048d15 << flag, - 0x00123456 << flag, 0x0048d159 << flag, - 0x01234567 << flag, 0x048d159e << flag, - 0x12345678 << flag, 0x48d159e2 << flag, - 0x23456789 << flag, 0x8d159e26 << flag, - 0x3456789a << flag, 0xd159e26a << flag, - 0x456789ab << flag, 0x159e26af << flag, - 0x56789abc << flag, 0x59e26af3 << flag, - 0x6789abcd << flag, 0x9e26af37 << flag, - 0x789abcde << flag, 0xe26af37b << flag, - 0x89abcdef << flag, 0x26af37bc << flag, - 0x9abcdef0 << flag, 0x6af37bc3 << flag, - 0xabcdef0f << flag); - - /* Without this check, a stub makecontext can make us spin forever. */ - if (memcmp (&tempctx, &ctx[1], sizeof ctx[1]) == 0) - { - puts ("makecontext was a no-op, presuming not implemented"); - return 0; - } - } + errno = 0; + makecontext (&ctx[1], (void (*) (void)) f1, 33, + 0x00000001 << flag, 0x00000004 << flag, + 0x00000012 << flag, 0x00000048 << flag, + 0x00000123 << flag, 0x0000048d << flag, + 0x00001234 << flag, 0x000048d1 << flag, + 0x00012345 << flag, 0x00048d15 << flag, + 0x00123456 << flag, 0x0048d159 << flag, + 0x01234567 << flag, 0x048d159e << flag, + 0x12345678 << flag, 0x48d159e2 << flag, + 0x23456789 << flag, 0x8d159e26 << flag, + 0x3456789a << flag, 0xd159e26a << flag, + 0x456789ab << flag, 0x159e26af << flag, + 0x56789abc << flag, 0x59e26af3 << flag, + 0x6789abcd << flag, 0x9e26af37 << flag, + 0x789abcde << flag, 0xe26af37b << flag, + 0x89abcdef << flag, 0x26af37bc << flag, + 0x9abcdef0 << flag, 0x6af37bc3 << flag, + 0xabcdef0f << flag); + + /* Without this check, a stub makecontext can make us spin forever. */ + if (errno == ENOSYS) + { + puts ("makecontext not implemented"); + back_in_main = 1; + return 0; + } /* Play some tricks with this context. */ if (++global == 1) |