Issue 7909 - test039 alignment issue
Summary: test039 alignment issue
Status: VERIFIED FIXED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: slapd (show other issues)
Version: unspecified
Hardware: All All
: --- normal
Target Milestone: ---
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-07-29 16:27 UTC by richton@nbcs.rutgers.edu
Modified: 2014-08-01 21:05 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description richton@nbcs.rutgers.edu 2014-07-29 16:27:03 UTC
Full_Name: Aaron Richton
Version: RE24
OS: Solaris 9
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (128.6.31.135)


test039 reproducibly reveals a misaligned access.

Current function is slap_writewait_play
  294                           sc->sc_writewait( op, sc );
(dbx) where
current thread: t@14
=>[1] slap_writewait_play(op = 0x101c6ee00), line 294 in "result.c"
  [2] send_ldap_ber(op = 0x101c6ee00, ber = 0xffffffff70ffecd8), line 367 in
"result.c"
  [3] slap_send_search_entry(op = 0x101c6ee00, rs = 0xffffffff70fffaa8), line
1430 in "result.c"
  [4] monitor_send_children(op = 0x101c6ee00, rs = 0xffffffff70fffaa8,
e_nonvolatile = 0x1007fbc88, e_ch = (nil), sub = 1), line 124 in "search.c"
  [5] monitor_send_children(op = 0x101c6ee00, rs = 0xffffffff70fffaa8,
e_nonvolatile = 0x1007fa298, e_ch = (nil), sub = 1), line 139 in "search.c"
  [6] monitor_back_search(op = 0x101c6ee00, rs = 0xffffffff70fffaa8), line 255
in "search.c"
  [7] fe_op_search(op = 0x101c6ee00, rs = 0xffffffff70fffaa8), line 402 in
"search.c"
  [8] overlay_op_walk(op = 0x101c6ee00, rs = 0xffffffff70fffaa8, which =
op_search, oi = 0x10067a890, on = (nil)), line 671 in "backover.c"
  [9] over_op_func(op = 0x101c6ee00, rs = 0xffffffff70fffaa8, which =
op_search), line 723 in "backover.c"
  [10] over_op_search(op = 0x101c6ee00, rs = 0xffffffff70fffaa8), line 750 in
"backover.c"
  [11] do_search(op = 0x101c6ee00, rs = 0xffffffff70fffaa8), line 247 in
"search.c"
  [12] connection_operation(ctx = 0xffffffff70fffc20, arg_v = 0x101c6ee00), line
1155 in "connection.c"
  [13] ldap_int_thread_pool_wrapper(xpool = 0x10061c0e0), line 688 in "tpool.c"
(dbx) threads
      t@1  a  l@1   ?()   LWP suspended in  __lwp_wait() 
      t@2  a  l@2   slapd_daemon_task()   LWP suspended in  _poll() 
      t@3  a  l@3   ldap_int_thread_pool_wrapper()   sleep on 0x1006d6208  in 
__lwp_park() 
      t@4  a  l@4   ldap_int_thread_pool_wrapper()   LWP suspended in  _poll() 
      t@5  a  l@5   ldap_int_thread_pool_wrapper()   LWP suspended in 
__lwp_park() 
      t@6  a  l@6   ldap_int_thread_pool_wrapper()   sleep on 0x1006a91e0  in 
__lwp_park() 
      t@7  a  l@7   ldap_int_thread_pool_wrapper()   sleep on 0x1006d6208  in 
__lwp_park() 
      t@8  a  l@8   ldap_int_thread_pool_wrapper()   LWP suspended in 
__lwp_park() 
      t@9  a  l@9   ldap_int_thread_pool_wrapper()   LWP suspended in 
__lwp_park() 
     t@10  a l@10   ldap_int_thread_pool_wrapper()   sleep on 0x10081d9e0  in 
__lwp_park() 
     t@11  a l@11   ldap_int_thread_pool_wrapper()   sleep on 0x10081d9e0  in 
__lwp_park() 
     t@12  a l@12   ldap_int_thread_pool_wrapper()   LWP suspended in 
__lwp_park() 
     t@13  a l@13   ldap_int_thread_pool_wrapper()   sleep on 0x1006d6208  in 
__lwp_park() 
o>   t@14  a l@14   ldap_int_thread_pool_wrapper()   signal SIGBUS in 
slap_writewait_play() 
     t@15  a l@15   ldap_int_thread_pool_wrapper()   LWP suspended in 
__lwp_park() 
     t@16  a l@16   ldap_int_thread_pool_wrapper()   sleep on 0x1006a91e0  in 
__lwp_park() 
     t@17  a l@17   ldap_int_thread_pool_wrapper()   LWP suspended in  _write()

     t@18  a l@18   ldap_int_thread_pool_wrapper()   LWP suspended in  _poll() 
Comment 1 richton@nbcs.rutgers.edu 2014-07-29 21:37:57 UTC
I should have noted that this occurs in the third slapd instance 
(slapd-glue-ldap.conf).

Comment 2 Howard Chu 2014-07-30 02:18:56 UTC
richton@nbcs.rutgers.edu wrote:
> Full_Name: Aaron Richton
> Version: RE24
> OS: Solaris 9
> URL: ftp://ftp.openldap.org/incoming/
> Submission from: (NULL) (128.6.31.135)
>
>
> test039 reproducibly reveals a misaligned access.
>
> Current function is slap_writewait_play
>    294                           sc->sc_writewait( op, sc );

Please show the output for
	print *sc

Thanks.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/

Comment 3 Howard Chu 2014-07-30 07:24:03 UTC
changed notes
changed state Open to Test
moved from Incoming to Development
Comment 4 Quanah Gibson-Mount 2014-07-30 10:22:53 UTC
changed notes
changed state Test to Release
Comment 5 richton@nbcs.rutgers.edu 2014-07-30 11:39:08 UTC
On Tue, 29 Jul 2014, Howard Chu wrote:

> 	print *sc

*sc = {
     sc_next      = 0xffffffff70fff3d8
     sc_response  = 0x1002b3330 = &`slapd`rwm.c`rwm_swap_attrs(Operation *op, SlapReply *rs)
     sc_cleanup   = 0x1002b0c18 = &`slapd`rwm.c`rwm_op_cleanup(Operation *op, SlapReply *rs)
     sc_writewait = 0x11
     sc_private   = 0x101840610
}

Comment 6 Howard Chu 2014-07-30 13:56:40 UTC
Aaron Richton wrote:
> On Tue, 29 Jul 2014, Howard Chu wrote:
>
>> 	print *sc
>
> *sc = {
>       sc_next      = 0xffffffff70fff3d8
>       sc_response  = 0x1002b3330 = &`slapd`rwm.c`rwm_swap_attrs(Operation *op, SlapReply *rs)
>       sc_cleanup   = 0x1002b0c18 = &`slapd`rwm.c`rwm_op_cleanup(Operation *op, SlapReply *rs)
>       sc_writewait = 0x11
>       sc_private   = 0x101840610
> }
>
>
Thanks, fixed in master.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/

Comment 7 Howard Chu 2014-07-31 10:30:40 UTC
changed notes
Comment 8 Howard Chu 2014-07-31 10:31:25 UTC
changed notes
changed state Release to Closed
Comment 9 Michael Ströder 2014-07-31 17:10:08 UTC
What's the status of this in RE24 branch? The ITS says released in RE24.

But AFAICS there are three related commits but the second *removes* the
ITS#7909 comment from CHANGES while the modification to rwm.c is still there.

Ciao, Michael.

commit 2d7a796670430f10745c3e59b05ef64c29274206
Author: Quanah Gibson-Mount <quanah@openldap.org>
Date:   Wed Jul 30 13:44:26 2014 -0500

    Remove 7909, not released

commit 2507da671d518301ca7a9ea2fcbdc4b5fe464d6a
Author: Quanah Gibson-Mount <quanah@openldap.org>
Date:   Wed Jul 30 12:22:37 2014 -0500

    ITS#7909

commit 067d9d55ae5b6541046ec446a011058b5b6aabc9
Author: Howard Chu <hyc@openldap.org>
Date:   Wed Jul 30 06:55:36 2014 -0700

    ITS#7909 fix slap_callback init


Comment 10 Howard Chu 2014-07-31 17:20:31 UTC
michael@stroeder.com wrote:
> What's the status of this in RE24 branch? The ITS says released in RE24.
>
> But AFAICS there are three related commits but the second *removes* the
> ITS#7909 comment from CHANGES while the modification to rwm.c is still there.

ITS#7909 was due to the change in ITS#7904. Since this was not a bug in 
already-released code it does not get tracked as its own ITS.
>
> Ciao, Michael.
>
> commit 2d7a796670430f10745c3e59b05ef64c29274206
> Author: Quanah Gibson-Mount <quanah@openldap.org>
> Date:   Wed Jul 30 13:44:26 2014 -0500
>
>      Remove 7909, not released
>
> commit 2507da671d518301ca7a9ea2fcbdc4b5fe464d6a
> Author: Quanah Gibson-Mount <quanah@openldap.org>
> Date:   Wed Jul 30 12:22:37 2014 -0500
>
>      ITS#7909
>
> commit 067d9d55ae5b6541046ec446a011058b5b6aabc9
> Author: Howard Chu <hyc@openldap.org>
> Date:   Wed Jul 30 06:55:36 2014 -0700
>
>      ITS#7909 fix slap_callback init
>
>
>
>


-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/

Comment 11 Michael Ströder 2014-07-31 17:20:45 UTC
michael@stroeder.com wrote:
> What's the status of this in RE24 branch? The ITS says released in RE24.
> 
> But AFAICS there are three related commits but the second *removes* the
> ITS#7909 comment from CHANGES while the modification to rwm.c is still there.

s/second/last/

> commit 2d7a796670430f10745c3e59b05ef64c29274206
> Author: Quanah Gibson-Mount <quanah@openldap.org>
> Date:   Wed Jul 30 13:44:26 2014 -0500
> 
>     Remove 7909, not released
> 
> commit 2507da671d518301ca7a9ea2fcbdc4b5fe464d6a
> Author: Quanah Gibson-Mount <quanah@openldap.org>
> Date:   Wed Jul 30 12:22:37 2014 -0500
> 
>     ITS#7909
> 
> commit 067d9d55ae5b6541046ec446a011058b5b6aabc9
> Author: Howard Chu <hyc@openldap.org>
> Date:   Wed Jul 30 06:55:36 2014 -0700
> 
>     ITS#7909 fix slap_callback init

Comment 12 OpenLDAP project 2014-08-01 21:05:03 UTC
due to #7904
fixed in master
fixed in RE24/RE25