Uploaded image for project: 'Couchbase C client library libcouchbase'
  1. Couchbase C client library libcouchbase
  2. CCBC-1205

lcb_errctx_kv_endpoint returning wrong length for the string

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.0.0
    • 3.0.2
    • None
    • None
    • 1

    Description

      So – it seems to be including the null terminator at the end.  It appears like maybe the returned length is just the full length of the fixed length char array?  But I could be wrong - just a guess.  

      Attachments

        For Gerrit Dashboard: CCBC-1205
        # Subject Branch Project Status CR V
        129197,2 CCBC-1205: do not include trailing zero for endpoint length for KV context master libcouchbase Status: MERGED +2 +1

        Activity

          Ascending order - Click to sort in descending order
          avsej Sergey Avseyev added a comment -

          Yes, the implementation of the endpoint right now is char array inside the structure. Do you want it to return one character less so that the caller wont see the null terminator?

          avsej Sergey Avseyev added a comment - Yes, the implementation of the endpoint right now is char array inside the structure. Do you want it to return one character less so that the caller wont see the null terminator?
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-7.0.0-3115 contains libcouchbase commit 7a5e905 with commit message:
          CCBC-1205: do not include trailing zero for endpoint length for KV context

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-3115 contains libcouchbase commit 7a5e905 with commit message: CCBC-1205 : do not include trailing zero for endpoint length for KV context

          People

            avsej Sergey Avseyev
            david.kelly David Kelly
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty