Bug 239764

Summary: bitwise matching plugin
Product: [Retired] 389 Reporter: Andrew Bartlett <abartlet>
Component: Database - Indexes/SearchesAssignee: Noriko Hosoi <nhosoi>
Status: CLOSED CURRENTRELEASE QA Contact: Viktor Ashirov <vashirov>
Severity: medium Docs Contact:
Priority: medium    
Version: 1.1.0betaCC: amsharma, edewata, rmeggins
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-07 16:47:17 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 427409    
Attachments:
Description Flags
Patch to implement bitwise operations
none
cvs commit log none

Description Andrew Bartlett 2007-05-11 06:26:44 UTC
Description of problem:
Samba requires backend LDAP servers to support bitwise operations


Version-Release number of selected component (if applicable):
1.1 CVS

How reproducible:
Every time


Steps to Reproduce:
1. Search with filters like
(|(|(&(!(groupType:1.2.840.113556.1.4.803:=1))(groupType:1.2
.840.113556.1.4.803:=2147483648)(groupType:1.2.840.113556.1.4.804:=10))(samAccountType=805306368))(samAccountType=805306369))

2. 
3.
  
Actual results:
No match on the extended matching rule

Expected results:
Valid matches

Additional info:

This patch works for Samba's tests, but hasn't been more broadly or specifically
tested

Comment 1 Andrew Bartlett 2007-05-11 06:26:44 UTC
Created attachment 154518 [details]
Patch to implement bitwise operations

Comment 2 Noriko Hosoi 2007-05-11 16:38:45 UTC
Your implementation looks good. 

Comment 3 Rich Megginson 2007-05-16 19:48:24 UTC
Created attachment 154865 [details]
cvs commit log

I added an --enable-bitwise option to configure (on by default - use
--disable-bitwise to disable it).  I also added the plugin entry to the
configuration during new instance creation.

Comment 4 Andrew Bartlett 2007-07-25 01:22:27 UTC
Samba4 uses this nicely, what do we need to wait for to close this bug as fixed?

Comment 5 Rich Megginson 2007-07-25 14:42:39 UTC
(In reply to comment #4)
> Samba4 uses this nicely, what do we need to wait for to close this bug as fixed?

It takes a QA person to move this to the CLOSED/RESOLVED state.  All us poor
developers can do is to move it to the MODIFIED state, which means some lucky QA
person has the task of verifying this.

Unfortunately, we have no QA directly assigned to Fedora DS.

So, for all intents and purposes, we can treat this bug as resolved since
bitwise is included in Fedora DS and works for its intended purpose.

Comment 7 Chandrasekar Kannan 2009-06-22 23:44:06 UTC
(In reply to comment #0)
> Additional info:
> 
> This patch works for Samba's tests, but hasn't been more broadly or specifically
> tested  

Andrew - can you share or provide some information about where to find/view these samba tests ..

Comment 8 Andrew Bartlett 2010-03-12 09:49:28 UTC
Endi,

I take it that now Fedora DS now works as a Samba backend that this is all done?

Certainly, the LDAP tests I'm describing are now part of the standard testsuite, and that is run against Fedord DS (389).

Comment 9 Endi Sukma Dewata 2010-03-12 17:25:17 UTC
Hi Andrew,

We have a machine called buildsamba02 in the Samba build farm that is configured to run with DS backend. These are the test results:

http://build.samba.org/?function=View+Build;host=buildsamba02;tree=samba_4_0_test;compiler=gcc

As of today there are still about ~80 failures. Do you know which test cases involve bitwise operations?

Comment 11 Rich Megginson 2011-06-16 14:08:42 UTC
This is covered by the filter/bitw_filter.sh test.

Comment 12 Amita Sharma 2011-06-17 05:58:18 UTC
filter run Tests  PASS       : 100% (257/257)

Marking VERIFIED sanity.