Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.2
    • Fix Version/s: 1.1.3
    • Component/s: library
    • Security Level: Public
    • Labels:
      None

      Description

      the ctx struct's freed before the exception ( exc ) is raised - so it references invalid memory

      ext/couchbase_ext/couchbase_ext.c- exc = ctx->exception; <----
      ext/couchbase_ext/couchbase_ext.c- extended = ctx->extended;
      ext/couchbase_ext/couchbase_ext.c: free(ctx); <------
      ext/couchbase_ext/couchbase_ext.c- if (exc != Qnil)

      { ext/couchbase_ext/couchbase_ext.c- rb_exc_raise(exc); <------ ext/couchbase_ext/couchbase_ext.c- }
      # Subject Project Status CR V
      For Gerrit Dashboard: &For+RCBC-60=message:RCBC-60

        Activity

        james.mauss James Mauss created issue -
        Hide
        avsej Sergey Avseyev added a comment -

        free(ctx) means deallocate the memory of the struct. ctx->exception is just a pointer, and it is valid

        Show
        avsej Sergey Avseyev added a comment - free(ctx) means deallocate the memory of the struct. ctx->exception is just a pointer, and it is valid
        Hide
        avsej Sergey Avseyev added a comment -

        It would be better if I'll get stacktrace or minimal working script (or both)

        Show
        avsej Sergey Avseyev added a comment - It would be better if I'll get stacktrace or minimal working script (or both)
        avsej Sergey Avseyev made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.1.3 [ 10361 ]
        Resolution Fixed [ 1 ]
        avsej Sergey Avseyev made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        brett19 Brett Lawson made changes -
        Workflow jira [ 18557 ] Couchbase SDK Workflow [ 44848 ]

          People

          • Assignee:
            avsej Sergey Avseyev
            Reporter:
            james.mauss James Mauss
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes