[bugzilla-daemon@xxxxxxxxxxxxx]

Comment # 6 on bug 8841 from hustpigeon

(In reply to comment #5)
> This code you copy pasted does exactly what is described in X.691 chapters
> 10.9.3.6 and 10.9.3.7 and put in length variable the content of the "length"
> fields as drawn in the spec.

so, dissect_per_normally_small_nonnegative_whole_number has a bug,right?

because function int dissect_per_length_determinant(******, length) 's
parameter length is lenght not value.

do you agree this is a bug? 
thank you very much
///////////////////////////////////////////////////////////////////////
static guint32
dissect_per_normally_small_nonnegative_whole_number(tvbuff_t *tvb, guint32
offset, asn1_ctx_t *actx, proto_tree *tree, int hf_index, guint32 *length)
{


(1)--->    if(!small_number)//
    {

       offset=dissect_per_boolean(tvb, offset, actx, tree, -1, 
     *length<<=1;
     if (length_bit) 
(value)     {
(2)------>   *length|=1;--->it is value not length
     }

    }

(length)    /* 10.6.2 */
(3)------> offset=dissect_per_length_determinant( "..." ,hf_index, 
                                                 length -->it is length not
value)

    return offset;
}

---

      You are receiving this mail because:

• You are watching all bug changes.