Changeset 526

Show
Ignore:
Timestamp:
12/17/06 18:26:21 (2 years ago)
Author:
alo
Message:

--

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • cherokee/trunk/ChangeLog

    r524 r526  
     12006-12-17  Alvaro Lopez Ortega  <alvaro@alobbs.com> 
     2 
     3        * cherokee/validator_htpasswd.c (cherokee_validator_htpasswd_check): 
     4        Better initialization.  It makes the validator safer is the 
     5        password file is empty. 
     6 
     7        * cherokee/handler_common.c (stat_file): Better initialization. 
     8 
     9        * cherokee/handler_fcgi.h: post_len property is now size_t instead 
     10        cuint_t. 
     11 
     12        * cherokee/handler_scgi.c (build_header): Changed %d by FMT_OFFSET 
     13        for printing to post length. 
     14 
     15        * cherokee/server.c (cherokee_server_init): Check chdir() returned 
     16        value. 
     17 
     18        * cherokee/server.c (cherokee_server_write_pidfile): Check 
     19        fwrite() returned cdde. 
     20 
     21        * cherokee/connection.c (cherokee_connection_log_delayed): It 
     22        could return an uninitialized value under some circunstances. 
     23        Fixed. 
     24 
     25        * cherokee/socket.h (OPENSSL_LAST_ERROR): Better initialization. 
     26        Now it's a bit safer. 
     27 
     28        * cherokee/buffer.c (cherokee_buffer_read_from_fd): Changed "%u" 
     29        by FMT_OFFSET to print a size_t. 
     30        (cherokee_buffer_print_debug): variables hex_text, and ascii_text 
     31        weren't initilizated. 
     32 
     33        * cherokee/version.c (cherokee_version_add) 
     34        (cherokee_version_add_w_port): Compilation warning fixd: 'ret' may 
     35        be used uninitialized in this function 
     36 
     37        * cherokee/util.c (cherokee_estimate_va_length): Compilation 
     38        warning fixed. util.c:611: warning: value computed is not used 
     39 
    1402006-12-15  Alvaro Lopez Ortega  <alvaro@alobbs.com> 
    241 

  • cherokee/trunk/cherokee/buffer.c

    r514 r526  
    609609                } 
    610610 
    611                 PRINT_ERROR ("ERROR: read(%d, %u,..) -> errno=%d '%s'\n", fd, size, errno, strerror(errno)); 
     611                PRINT_ERROR ("ERROR: read(%d, " FMT_OFFSET ",..) -> errno=%d '%s'\n", fd, size, errno, strerror(errno)); 
    612612                return ret_error; 
    613613        } 
    …… 
    646646cherokee_buffer_print_debug (cherokee_buffer_t *buf, int len) 
    647647{ 
    648         int           i, length; 
    649         char          text[67]; 
    650         char*         hex_text; 
    651         char*         ascii_text; 
    652         unsigned char tmp; 
     648        int            i, length; 
     649        char          text[67]; 
     650        unsigned char  tmp; 
     651        char          *hex_text   = NULL; 
     652        char          *ascii_text = NULL; 
    653653 
    654654        if ((len == -1) || (buf->len <= len)) { 

  • cherokee/trunk/cherokee/connection.c

    r466 r526  
    19041904cherokee_connection_log_delayed (cherokee_connection_t *conn) 
    19051905{ 
    1906         ret_t ret; 
     1906        ret_t ret = ret_ok; 
    19071907 
    19081908        if ((conn->log_at_end) && (conn->logger_ref)) { 

  • cherokee/trunk/cherokee/handler_common.c

    r476 r526  
    101101           char *path, cherokee_iocache_entry_t **io_entry, struct stat **info) 
    102102{        
    103         int   re; 
    104103        ret_t ret; 
     104        int   re  = -1; 
    105105 
    106106        /* Without cache 

  • cherokee/trunk/cherokee/handler_fcgi.h

    r476 r526  
    4848        cherokee_socket_t             socket; 
    4949        cherokee_handler_fcgi_post_t  post_phase; 
    50         cuint_t                       post_len; 
     50        size_t                        post_len; 
    5151        cherokee_buffer_t             write_buffer; 
    5252} cherokee_handler_fcgi_t; 

  • cherokee/trunk/cherokee/handler_scgi.c

    r501 r526  
    240240        char    tmp[64]; 
    241241 
    242         len = snprintf (tmp, 64, "%d", hdl->post_len); 
     242        len = snprintf (tmp, 64, FMT_OFFSET, hdl->post_len); 
    243243         
    244244        set_env (HDL_CGI_BASE(hdl), "CONTENT_LENGTH", tmp, len); 

  • cherokee/trunk/cherokee/server.c

    r506 r526  
    890890cherokee_server_init (cherokee_server_t *srv)  
    891891{    
     892        int            re; 
    892893        ret_t          ret; 
    893894        struct passwd *ent; 
    …… 
    979980        /* Change current directory 
    980981         */ 
    981         chdir ("/"); 
     982        re = chdir ("/"); 
     983        if (re < 0) { 
     984                PRINT_ERROR ("Couldn't chmod(\"/\"): %s\n", strerror(errno)); 
     985                return ret_error; 
     986        } 
    982987 
    983988        /* Create the threads 
    …… 
    17691774cherokee_server_write_pidfile (cherokee_server_t *srv) 
    17701775{ 
    1771         FILE *file; 
     1776        size_t  written; 
     1777        FILE   *file; 
    17721778        CHEROKEE_TEMP(buffer, 10); 
    17731779 
    …… 
    17821788 
    17831789        snprintf (buffer, buffer_size, "%d\n", getpid()); 
    1784         fwrite (buffer, 1, strlen(buffer), file); 
     1790        written = fwrite (buffer, 1, strlen(buffer), file); 
    17851791        fclose (file); 
    17861792 
    1787         return ret_ok; 
    1788 } 
    1789  
     1793        if (written <= 0) 
     1794                return ret_error; 
     1795 
     1796        return ret_ok; 
     1797} 
     1798 

  • cherokee/trunk/cherokee/socket.h

    r466 r526  
    164164 
    165165#ifdef HAVE_OPENSSL 
    166 # define OPENSSL_LAST_ERROR(error)  do { int n;                              \ 
    167                                          while ((n = ERR_get_error()))       \ 
    168                                           error = ERR_error_string(n, NULL); \ 
     166# define OPENSSL_LAST_ERROR(error)  do { int n;                                \ 
     167                                         error = "unknown";                    \ 
     168                                         while ((n = ERR_get_error()))         \ 
     169                                            error = ERR_error_string(n, NULL); \ 
    169170                                    } while (0) 
    170171#endif 

  • cherokee/trunk/cherokee/util.c

    r493 r526  
    609609                        goto reswitch; 
    610610                case 'c': 
    611                         va_arg(ap, int); 
     611                        (void) va_arg(ap, int); 
    612612                        len++; 
    613613                        break; 

  • cherokee/trunk/cherokee/validator_htpasswd.c

    r476 r526  
    283283        } 
    284284 
    285         ret = ret_error; 
     285        ret      = ret_error; 
     286        ret_auth = ret_error; 
     287 
    286288        while (!feof(f)) { 
    287289                if (fgets (line, line_size, f) == NULL) 

  • cherokee/trunk/cherokee/version.c

    r493 r526  
    4343        case cherokee_version_full: 
    4444                ret = cherokee_buffer_add_str (buf, "Cherokee web server " PACKAGE_MAJOR_VERSION "." PACKAGE_MINOR_VERSION "." PACKAGE_MICRO_VERSION " (" OS_TYPE ")"); 
     45        default: 
     46                ret = ret_error; 
    4547        } 
    4648 
    …… 
    6567        case cherokee_version_full: 
    6668                ret = cherokee_buffer_add_va (buf, "Cherokee web server Port %d " PACKAGE_MAJOR_VERSION "." PACKAGE_MINOR_VERSION "." PACKAGE_MICRO_VERSION " (" OS_TYPE ")", port); 
     69        default: 
     70                ret = ret_error; 
    6771        } 
    6872