#7 Adding Acct-Seesion-ID to Access-Request (auth_radius)

trunk
closed-accepted
modules (179)
5
2008-08-30
2008-08-11
Dror Wald
No

This patch to auth_radius module adds Acct-Session-Id to Access-Request According to RFC2866:
"
5.5. Acct-Session-Id
Description
This attribute is a unique Accounting ID to make it easy to match
start and stop records in a log file. The start and stop records
for a given session MUST have the same Acct-Session-Id. An
Accounting-Request packet MUST have an Acct-Session-Id. An
Access-Request packet MAY have an Acct-Session-Id; if it does,
then the NAS MUST use the same Acct-Session-Id in the Accounting-
Request packets for that session.
"

The purpose of this patch is to create a relationship between an auth and acct request, based on SIP CALL-ID value (Acct-Session-Id attribute).

Discussion

  • Dror Wald

    Dror Wald - 2008-08-11

    Logged In: YES
    user_id=2176865
    Originator: YES

    File Added: authrad_mod.c.diff

     
  • Dror Wald

    Dror Wald - 2008-08-13

    authrad_mod.c.diff (based on version 1.4.1)

     
  • Dror Wald

    Dror Wald - 2008-08-13

    Logged In: YES
    user_id=2176865
    Originator: YES

    File Added: authrad_mod.c.diff

     
  • Dror Wald

    Dror Wald - 2008-08-13
    • milestone: --> 869101
    • summary: Adding Acct-Seesion-ID to Access-Request --> Adding Acct-Seesion-ID to Access-Request (auth_radius)
     
  • Dror Wald

    Dror Wald - 2008-08-13

    Logged In: YES
    user_id=2176865
    Originator: YES

    File Added: sterman.c.diff

     
  • Dror Wald

    Dror Wald - 2008-08-13

    sterman.c.diff (based on version 1.4.1)

     
  • Bogdan-Andrei Iancu

    • milestone: 869101 --> trunk
    • assigned_to: nobody --> bogdan_iancu
    • status: open --> open-accepted
     
  • Bogdan-Andrei Iancu

    Logged In: YES
    user_id=1275325
    Originator: NO

    Hi Dror,

    I applied your patch on the trunk, but with some small changes: CALLID is mandatory in SIP, so, instead of setting "none", an error is generated. And there is no need to allocate separate pkg memory for the value - anthow, in the original version, that was a mem leak.

    Thanks and regards,
    Bogdan

     
  • Bogdan-Andrei Iancu

    • status: open-accepted --> closed-accepted
     

Log in to post a comment.