s3:share_mode_lock: remove unused reproducer for bug #14428
authorStefan Metzmacher <metze@samba.org>
Fri, 28 Aug 2020 14:28:41 +0000 (16:28 +0200)
committerVolker Lendecke <vl@samba.org>
Mon, 31 Aug 2020 13:34:17 +0000 (13:34 +0000)
BUG: https://bugzilla.samba.org/show_bug.cgi?id=14428

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Mon Aug 31 13:34:17 UTC 2020 on sn-devel-184

source3/locking/share_mode_lock.c
source3/script/tests/test_durable_handle_reconnect.sh

index 6588f09b285bff6b0af785747caacdc56bac345f..1c4d3a422214b2afbc23066f93f896622dbac4fe 100644 (file)
@@ -1783,20 +1783,6 @@ bool share_mode_cleanup_disconnected(struct file_id fid,
                goto done;
        }
 
-       /*
-        * This is a temporary reproducer for the origin of
-        * https://bugzilla.samba.org/show_bug.cgi?id=14428
-        * "PANIC: assert failed in get_lease_type()"
-        *
-        * This will be removed again once the bug is demonstrated
-        * and fixed.
-        */
-       TALLOC_FREE(state.lck);
-       state.lck = get_existing_share_mode_lock(frame, fid);
-       if (state.lck != NULL) {
-               DBG_ERR("Reproduced BUG#14428\n");
-       }
-
        ret = true;
 done:
        talloc_free(frame);
index 66a35f08f17edf0dd62c2d70c79c9449c3de0429..2f6b308eebe98513b26e2aa477b83fb3f235accc 100755 (executable)
@@ -23,20 +23,12 @@ if [ $SMBD_DONT_LOG_STDOUT -eq 1 ]; then
        SMBD_LOG_FILES="$SMBD_LOG_FILES $_SMBD_LOG_FILE"
 fi
 
-bug_count_0=$(grep 'Reproduced BUG#14428' $SMBD_LOG_FILES | wc -l)
-
 testit "durable_v2_delay.durable_v2_reconnect_delay_msec" $VALGRIND \
        $BINDIR/smbtorture //$SERVER_IP/durable \
        -U$USERNAME%$PASSWORD \
        smb2.durable-v2-delay.durable_v2_reconnect_delay_msec ||
        failed=$(expr $failed + 1)
 
-bug_count_1=$(grep 'Reproduced BUG#14428' $SMBD_LOG_FILES | wc -l)
-
-testit "bug.14428 bug_count_0[$bug_count_0] bug_count_1[$bug_count_1]" \
-       test $bug_count_0 -eq $bug_count_1 ||
-       failed=$(expr $failed + 1)
-
 rm $delay_inject_conf
 
 testok $0 $failed