Navigation

Changeset 403:26aafbbc1640 in freeDiameter for extensions/app_radgw/rgwx_auth.c


Ignore:
Timestamp:
Jul 8, 2010, 2:24:19 PM (14 years ago)
Author:
Sebastien Decugis <sdecugis@nict.go.jp>
Branch:
default
Phase:
public
Message:

Cleanup all compilation warnings in base code for 32 bit arch

File:
1 edited

Legend:

Unmodified
Added
Removed

  • extensions/app_radgw/rgwx_auth.c

    r388 r403  
    236236        const char * prefix = "Diameter/";
    237237        size_t pref_len;
    238         char * dh = NULL;
     238        uint8_t * dh = NULL;
    239239        size_t dh_len = 0;
    240         char * dr = NULL;
     240        uint8_t * dr = NULL;
    241241        size_t dr_len = 0;
    242         char * si = NULL;
     242        uint8_t * si = NULL;
    243243        size_t si_len = 0;
    244         char * un = NULL;
     244        uint8_t * un = NULL;
    245245        size_t un_len = 0;
    246         uint32_t status_type;
    247246        size_t nattr_used = 0;
    248247        struct avp ** avp_tun = NULL, *avp = NULL;
    … …  
    316315        for (idx = 0; idx < rad_req->attr_used; idx++) {
    317316                struct radius_attr_hdr * attr = (struct radius_attr_hdr *)(rad_req->buf + rad_req->attr_pos[idx]);
    318                 char * attr_val = (char *)(attr + 1);
     317                uint8_t * attr_val = (uint8_t *)(attr + 1);
    319318                size_t attr_len = attr->length - sizeof(struct radius_attr_hdr);
    320319               
    … …  
    343342                        case RADIUS_ATTR_STATE:
    344343                                if ((attr_len > pref_len + 5 /* for the '/'s and non empty strings */ )
    345                                         && ! strncmp(attr_val, prefix, pref_len)) { /* should we make it strncasecmp? */
     344                                        && ! memcmp(attr_val, prefix, pref_len)) {
    346345                                        int i, start;
    347346
    … …  
    376375               
    377376                        case RADIUS_ATTR_USER_NAME:
    378                                 TRACE_DEBUG(ANNOYING, "Found a User-Name attribute: '%.*s'", attr_len, attr_len ? attr_val : "");
     377                                TRACE_DEBUG(ANNOYING, "Found a User-Name attribute: '%.*s'", attr_len, attr_len ? (char *)attr_val : "");
    379378                                un = attr_val;
    380379                                un_len = attr_len;
    … …  
    430429                if (i <= 0) {
    431430                        /* Not found in the User-Name => we use the local domain of this gateway */
    432                         value.os.data = fd_g_config->cnf_diamrlm;
     431                        value.os.data = (uint8_t *)fd_g_config->cnf_diamrlm;
    433432                        value.os.len  = fd_g_config->cnf_diamrlm_len;
    434433                } else {
    … …  
    455454                if (si_len) {
    456455                        /* We already have the Session-Id, just use it */
    457                         CHECK_FCT( fd_sess_fromsid ( si, si_len, session, NULL) );
     456                        CHECK_FCT( fd_sess_fromsid ( (char *) /* this cast will be removed later */ si, si_len, session, NULL) );
    458457                } else {
    459458                        /* Create a new Session-Id string */
    … …  
    544543                if (got_empty_eap) {
    545544                        value.os.len = 0;
    546                         value.os.data = "";
     545                        value.os.data = (uint8_t *)"";
    547546                } else {
    548547                        CHECK_MALLOC( value.os.data = radius_msg_get_eap(rad_req, &value.os.len) );
    … …  
    10661065        struct avp *avp, *next, *avp_x, *avp_y, *asid, *aoh;
    10671066        struct avp_hdr *ahdr, *sid, *oh;
    1068         char buf[254]; /* to store some attributes values (with final '\0') */
     1067        uint8_t buf[254]; /* to store some attributes values (with final '\0') */
     1068        size_t sz;
    10691069        int ta_set = 0;
    10701070        int no_str = 0; /* indicate if an STR is required for this server */
    … …  
    11891189        /* Now, save the session-id and eventually server info in a STATE or CLASS attribute */
    11901190        if ((*rad_fw)->hdr->code == RADIUS_CODE_ACCESS_CHALLENGE) {
    1191                 if (sizeof(buf) < snprintf(buf, sizeof(buf), "Diameter/%.*s/%.*s/%.*s",
    1192                                 oh->avp_value->os.len,  oh->avp_value->os.data,
    1193                                 ahdr->avp_value->os.len,  ahdr->avp_value->os.data,
    1194                                 sid->avp_value->os.len, sid->avp_value->os.data)) {
     1191                if (sizeof(buf) < (sz = snprintf((char *)buf, sizeof(buf), "Diameter/%.*s/%.*s/%.*s",
     1192                                oh->avp_value->os.len,  (char *)oh->avp_value->os.data,
     1193                                ahdr->avp_value->os.len,  (char *)ahdr->avp_value->os.data,
     1194                                sid->avp_value->os.len, (char *)sid->avp_value->os.data))) {
    11951195                        TRACE_DEBUG(INFO, "Data truncated in State attribute: %s", buf);
    11961196                }
    1197                 CONV2RAD_STR(RADIUS_ATTR_STATE, buf, strlen(buf), 0);
     1197                CONV2RAD_STR(RADIUS_ATTR_STATE, buf, sz, 0);
    11981198        }
    11991199
    12001200        if ((*rad_fw)->hdr->code == RADIUS_CODE_ACCESS_ACCEPT) {
    12011201                /* Add the Session-Id */
    1202                 if (sizeof(buf) < snprintf(buf, sizeof(buf), "Diameter/%.*s",
    1203                                 sid->avp_value->os.len, sid->avp_value->os.data)) {
     1202                if (sizeof(buf) < (sz = snprintf((char *)buf, sizeof(buf), "Diameter/%.*s",
     1203                                sid->avp_value->os.len, sid->avp_value->os.data))) {
    12041204                        TRACE_DEBUG(INFO, "Data truncated in Class attribute: %s", buf);
    12051205                }
    1206                 CONV2RAD_STR(RADIUS_ATTR_CLASS, buf, strlen(buf), 0);
     1206                CONV2RAD_STR(RADIUS_ATTR_CLASS, buf, sz, 0);
    12071207        }
    12081208       
    … …  
    15871587                                                        CHECK_FCT( fd_msg_avp_hdr ( inavp, &ahdr ) );
    15881588                                                       
    1589                                                         if (ahdr->avp_flags & AVP_FLAG_VENDOR == 0) {
     1589                                                        if ( ! (ahdr->avp_flags & AVP_FLAG_VENDOR)) {
    15901590                                                                switch (ahdr->avp_code) {
    15911591                                                                        case DIAM_ATTR_TUNNEL_TYPE:
    … …  
    16341634                                                                                        size_t pos;
    16351635                                                                                        int i;
    1636                                                                                         size_t buflen;
    16371636                                                                                        uint8_t * secret;       /* S */
    16381637                                                                                        size_t secret_len;
    … …  
    16531652                                                                                       
    16541653                                                                                        /* The plain text string P */
    1655                                                                                         CHECK_PARAM(ahdr->avp_value->os.len < 240);
     1654                                                                                        CHECK_PARAMS(ahdr->avp_value->os.len < 240);
    16561655                                                                                        buf[3] = ahdr->avp_value->os.len;
    16571656                                                                                        memcpy(&buf[4], ahdr->avp_value->os.data, ahdr->avp_value->os.len);
    … …  
    18171816                /* Add the auth-application-id required for STR, or 0 if no STR is required */
    18181817                CHECK_FCT( fd_msg_hdr( *diam_ans, &hdr ) );
    1819                 if (sizeof(buf) < snprintf(buf, sizeof(buf), CLASS_AAI_PREFIX "%u",
    1820                                 no_str ? 0 : hdr->msg_appl)) {
     1818                if (sizeof(buf) < (sz = snprintf((char *)buf, sizeof(buf), CLASS_AAI_PREFIX "%u",
     1819                                no_str ? 0 : hdr->msg_appl))) {
    18211820                        TRACE_DEBUG(INFO, "Data truncated in Class attribute: %s", buf);
    18221821                }
    1823                 CONV2RAD_STR(RADIUS_ATTR_CLASS, buf, strlen(buf), 0);
     1822                CONV2RAD_STR(RADIUS_ATTR_CLASS, buf, sz, 0);
    18241823        }
    18251824       
Note: See TracChangeset for help on using the changeset viewer.