Changeset 578


Ignore:
Timestamp:
Jun 25, 2011, 7:14:41 PM (14 years ago)
Author:
Herwig Bauernfeind
Message:

Update Samba 3.3 to 3.3.15 (security update)

Location:
branches/samba-3.3.x
Files:
25 edited

Legend:

Unmodified
Added
Removed

  • branches/samba-3.3.x/WHATSNEW.txt

    r491 r578  
     1                   ==============================
     2                   Release Notes for Samba 3.3.15
     3                         February 28, 2011
     4                   ==============================
     5
     6
     7This is a security release in order to address CVE-2011-0719.
     8
     9
     10o  CVE-2011-0719:
     11   All current released versions of Samba are vulnerable to
     12   a denial of service caused by memory corruption. Range
     13   checks on file descriptors being used in the FD_SET macro
     14   were not present allowing stack corruption. This can cause
     15   the Samba code to crash or to loop attempting to select
     16   on a bad file descriptor set.
     17
     18
     19Changes since 3.3.14
     20--------------------
     21
     22
     23o   Jeremy Allison <jra@samba.org>
     24    * BUG 7949: Fix DoS in Winbind and smbd with many file descriptors open.
     25
     26
     27######################################################################
     28Reporting bugs & Development Discussion
     29#######################################
     30
     31Please discuss this release on the samba-technical mailing list or by
     32joining the #samba-technical IRC channel on irc.freenode.net.
     33
     34If you do report problems then please try to send high quality
     35feedback. If you don't provide vital information to help us track down
     36the problem then you will probably be ignored.  All bug reports should
     37be filed under the Samba 3.3 product in the project's Bugzilla
     38database (https://bugzilla.samba.org/).
     39
     40
     41======================================================================
     42== Our Code, Our Bugs, Our Responsibility.
     43== The Samba Team
     44======================================================================
     45
     46
     47Release notes for older releases follow:
     48----------------------------------------
     49
    150                   ==============================
    251                   Release Notes for Samba 3.3.14
    … …  
    51100
    52101
    53 Release notes for older releases follow:
    54 ----------------------------------------
     102----------------------------------------------------------------------
     103
    55104
    56105                   ==============================

  • branches/samba-3.3.x/packaging/RHEL-CTDB/samba.spec

    r491 r578  
    66Packager: Samba Team <samba@samba.org>
    77Name:         samba
    8 Version:      3.3.14
     8Version:      3.3.15
    99Release:      ctdb.1
    1010Epoch:        0

  • branches/samba-3.3.x/packaging/RHEL/makerpms.sh

    r491 r578  
    2121USERID=`id -u`
    2222GRPID=`id -g`
    23 VERSION='3.3.14'
     23VERSION='3.3.15'
    2424REVISION=''
    2525SPECFILE="samba.spec"

  • branches/samba-3.3.x/packaging/RHEL/samba.spec

    r491 r578  
    66Packager: Samba Team <samba@samba.org>
    77Name:         samba
    8 Version:      3.3.14
     8Version:      3.3.15
    99Release:      1
    1010Epoch:        0

  • branches/samba-3.3.x/source/VERSION

    r491 r578  
    2626SAMBA_VERSION_MAJOR=3
    2727SAMBA_VERSION_MINOR=3
    28 SAMBA_VERSION_RELEASE=14
     28SAMBA_VERSION_RELEASE=15
    2929
    3030########################################################

  • branches/samba-3.3.x/source/client/client.c

    r221 r578  
    43804380 again:
    43814381
    4382         if (cli->fd == -1)
     4382        if (cli->fd < 0 || cli->fd >= FD_SETSIZE) {
     4383                errno = EBADF;
    43834384                return;
     4385        }
    43844386
    43854387        FD_ZERO(&fds);

  • branches/samba-3.3.x/source/client/dnsbrowse.c

    r206 r578  
    8080                if (fdset != NULL) {
    8181                        TALLOC_FREE(fdset);
     82                }
     83
     84                if (mdnsfd < 0 || mdnsfd >= FD_SETSIZE) {
     85                        errno = EBADF;
     86                        break;
    8287                }
    8388
    … …  
    184189                fdsetsz = howmany(mdnsfd + 1, NFDBITS) * sizeof(fd_mask);
    185190                fdset = TALLOC_ZERO(ctx, fdsetsz);
     191
     192                if (mdnsfd < 0 || mdnsfd >= FD_SETSIZE) {
     193                        errno = EBADF;
     194                        TALLOC_FREE(ctx);
     195                        return 1;
     196                }
     197
    186198                FD_SET(mdnsfd, fdset);
    187199

  • branches/samba-3.3.x/source/lib/events.c

    r221 r578  
    141141        struct fd_event *fde;
    142142
     143        if (fd < 0 || fd >= FD_SETSIZE) {
     144                errno = EBADF;
     145                return NULL;
     146        }
     147
    143148        if (!(fde = TALLOC_P(mem_ctx, struct fd_event))) {
    144149                return NULL;
    … …  
    191196
    192197        for (fde = event_ctx->fd_events; fde; fde = fde->next) {
     198                if (fde->fd < 0 || fde->fd >= FD_SETSIZE) {
     199                        /* We ignore here, as it shouldn't be
     200                           possible to add an invalid fde->fd
     201                           but we don't want FD_SET to see an
     202                           invalid fd. */
     203                        continue;
     204                }
     205
    193206                if (fde->flags & EVENT_FD_READ) {
    194207                        FD_SET(fde->fd, read_fds);

  • branches/samba-3.3.x/source/lib/packet.c

    r206 r578  
    107107        fd_set r_fds;
    108108
     109        if (ctx->fd < 0 || ctx->fd >= FD_SETSIZE) {
     110                errno = EBADF;
     111                return map_nt_error_from_unix(errno);
     112        }
     113
    109114        FD_ZERO(&r_fds);
    110115        FD_SET(ctx->fd, &r_fds);

  • branches/samba-3.3.x/source/lib/readline.c

    r206 r578  
    9292                timeout.tv_usec = 0;
    9393
     94                if (fd < 0 || fd >= FD_SETSIZE) {
     95                        errno = EBADF;
     96                        break;
     97                }
     98
    9499                FD_ZERO(&fds);
    95100                FD_SET(fd,&fds);

  • branches/samba-3.3.x/source/lib/select.c

    r206 r578  
    6666                        smb_panic("Could not create select pipe");
    6767
     68                if (select_pipe[0] < 0 || select_pipe[0] >= FD_SETSIZE) {
     69                        errno = EBADF;
     70                        return -1;
     71                }
     72
    6873                /*
    6974                 * These next two lines seem to fix a bug with the Linux
    … …  
    9297                FD_ZERO(readfds2);
    9398        }
     99
    94100        FD_SET(select_pipe[0], readfds2);
    95101

  • branches/samba-3.3.x/source/lib/util_sock.c

    r206 r578  
    961961
    962962        for (nread=0; nread < mincnt; ) {
     963                if (fd < 0 || fd >= FD_SETSIZE) {
     964                        errno = EBADF;
     965                        return map_nt_error_from_unix(EBADF);
     966                }
     967
    963968                FD_ZERO(&fds);
    964969                FD_SET(fd,&fds);
    … …  
    14931498        for (i=0; i<num_addrs; i++) {
    14941499                sockets[i] = socket(addrs[i].ss_family, SOCK_STREAM, 0);
    1495                 if (sockets[i] < 0)
     1500                if (sockets[i] < 0 || sockets[i] >= FD_SETSIZE)
    14961501                        goto done;
    14971502                set_blocking(sockets[i], false);
    … …  
    15421547
    15431548        for (i=0; i<num_addrs; i++) {
    1544                 if (sockets[i] == -1)
     1549                if (sockets[i] < 0 || sockets[i] >= FD_SETSIZE) {
     1550                        /* This cannot happen - ignore if so. */
    15451551                        continue;
     1552                }
    15461553                FD_SET(sockets[i], &wr_fds);
    15471554                FD_SET(sockets[i], &r_fds);

  • branches/samba-3.3.x/source/libaddns/dnssock.c

    r206 r578  
    219219                ssize_t ret;
    220220                int fd_ready;
    221                
     221
     222                if (fd < 0 || fd >= FD_SETSIZE) {
     223                        return ERROR_DNS_SOCKET_ERROR;
     224                }
     225
    222226                FD_ZERO( &rfds );
    223227                FD_SET( fd, &rfds );

  • branches/samba-3.3.x/source/libsmb/nmblib.c

    r206 r578  
    10981098        int ret;
    10991099
     1100        if (fd < 0 || fd >= FD_SETSIZE) {
     1101                errno = EBADF;
     1102                return NULL;
     1103        }
     1104
    11001105        FD_ZERO(&fds);
    11011106        FD_SET(fd,&fds);

  • branches/samba-3.3.x/source/nmbd/nmbd_packets.c

    r206 r578  
    16841684                count++;
    16851685
    1686         if((count*2) + 2 > FD_SETSIZE) {
     1686        if((count*2) + 2 >= FD_SETSIZE) {
    16871687                DEBUG(0,("create_listen_fdset: Too many file descriptors needed (%d). We can \
    16881688only use %d.\n", (count*2) + 2, FD_SETSIZE));
    … …  
    17001700
    17011701        /* Add in the broadcast socket on 137. */
     1702        if (ClientNMB < 0 || ClientNMB >= FD_SETSIZE) {
     1703                errno = EBADF;
     1704                SAFE_FREE(pset);
     1705                return True;
     1706        }
     1707
    17021708        FD_SET(ClientNMB,pset);
    17031709        sock_array[num++] = ClientNMB;
    … …  
    17061712        /* Add in the 137 sockets on all the interfaces. */
    17071713        for (subrec = FIRST_SUBNET; subrec; subrec = NEXT_SUBNET_EXCLUDING_UNICAST(subrec)) {
     1714                if (subrec->nmb_sock < 0 || subrec->nmb_sock >= FD_SETSIZE) {
     1715                        /* We have to ignore sockets outside FD_SETSIZE. */
     1716                        continue;
     1717                }
    17081718                FD_SET(subrec->nmb_sock,pset);
    17091719                sock_array[num++] = subrec->nmb_sock;
    … …  
    17121722
    17131723        /* Add in the broadcast socket on 138. */
     1724        if (ClientDGRAM < 0 || ClientDGRAM >= FD_SETSIZE) {
     1725                errno = EBADF;
     1726                SAFE_FREE(pset);
     1727                return True;
     1728        }
     1729
    17141730        FD_SET(ClientDGRAM,pset);
    17151731        sock_array[num++] = ClientDGRAM;
    … …  
    17181734        /* Add in the 138 sockets on all the interfaces. */
    17191735        for (subrec = FIRST_SUBNET; subrec; subrec = NEXT_SUBNET_EXCLUDING_UNICAST(subrec)) {
     1736                if (subrec->dgram_sock < 0 || subrec->dgram_sock >= FD_SETSIZE) {
     1737                        /* We have to ignore sockets outside FD_SETSIZE. */
     1738                        continue;
     1739                }
    17201740                FD_SET(subrec->dgram_sock,pset);
    17211741                sock_array[num++] = subrec->dgram_sock;
    … …  
    17681788#ifndef SYNC_DNS
    17691789        dns_fd = asyncdns_fd();
    1770         if (dns_fd != -1) {
     1790        if (dns_fd >= 0 && dns_fd < FD_SETSIZE) {
    17711791                FD_SET(dns_fd, &r_fds);
    17721792                maxfd = MAX( maxfd, dns_fd);

  • branches/samba-3.3.x/source/nsswitch/wb_common.c

    r206 r578  
    241241                switch (errno) {
    242242                        case EINPROGRESS:
     243
     244                                if (fd < 0 || fd >= FD_SETSIZE) {
     245                                        errno = EBADF;
     246                                        goto error_out;
     247                                }
     248
    243249                                FD_ZERO(&w_fds);
    244250                                FD_SET(fd, &w_fds);
    … …  
    384390                struct timeval tv;
    385391                fd_set r_fds;
    386                
     392
     393                if (winbindd_fd < 0 || winbindd_fd >= FD_SETSIZE) {
     394                        errno = EBADF;
     395                        winbind_close_sock();
     396                        return -1;
     397                }
     398
    387399                /* Catch pipe close on other end by checking if a read()
    388400                   call would not block by calling select(). */
    … …  
    444456                struct timeval tv;
    445457                fd_set r_fds;
    446                
     458
     459                if (winbindd_fd < 0 || winbindd_fd >= FD_SETSIZE) {
     460                        errno = EBADF;
     461                        winbind_close_sock();
     462                        return -1;
     463                }
     464
    447465                /* Catch pipe close on other end by checking if a read()
    448466                   call would not block by calling select(). */

  • branches/samba-3.3.x/source/printing/printing.c

    r256 r578  
    14131413        }
    14141414
     1415        if (pause_pipe[1] < 0 || pause_pipe[1] >= FD_SETSIZE) {
     1416                DEBUG(5,("start_background_queue: pipe fd out of range.\n"));
     1417                exit(1);
     1418        }
     1419
    14151420        background_lpq_updater_pid = sys_fork();
    14161421

  • branches/samba-3.3.x/source/smbd/dnsregister.c

    r370 r578  
    126126        if (dns_state->srv_ref != NULL) {
    127127                mdnsd_conn_fd = DNSServiceRefSockFD(dns_state->srv_ref);
     128                if (mdnsd_conn_fd < 0 || mdnsd_conn_fd >= FD_SETSIZE) {
     129                        return;
     130                }
    128131                FD_SET(mdnsd_conn_fd, listen_set);
    129132                return;
    … …  
    157160
    158161        mdnsd_conn_fd = DNSServiceRefSockFD(dns_state->srv_ref);
     162        if (mdnsd_conn_fd < 0 || mdnsd_conn_fd >= FD_SETSIZE) {
     163                return;
     164        }
    159165        FD_SET(mdnsd_conn_fd, listen_set);
    160166        *maxfd = MAX(*maxfd, mdnsd_conn_fd);

  • branches/samba-3.3.x/source/smbd/oplock.c

    r224 r578  
    242242{
    243243        if (koplocks) {
    244                 return koplocks->notification_fd;
     244                int fd = koplocks->notification_fd;
     245                if (fd < 0 || fd >= FD_SETSIZE) {
     246                        return -1;
     247                }
    245248        }
    246249

  • branches/samba-3.3.x/source/smbd/oplock_irix.c

    r206 r578  
    285285        }
    286286
     287        if (pfd[0] < 0 || pfd[0] >= FD_SETSIZE) {
     288                DEBUG(0,("setup_kernel_oplock_pipe: fd out of range.\n"));
     289                return False;
     290        }
     291
    287292        oplock_pipe_read = pfd[0];
    288293        oplock_pipe_write = pfd[1];

  • branches/samba-3.3.x/source/smbd/process.c

    r467 r578  
    699699static int select_on_fd(int fd, int maxfd, fd_set *fds)
    700700{
    701         if (fd != -1) {
     701        if (fd != -1 && fd < FD_SETSIZE) {
    702702                FD_SET(fd, fds);
    703703                maxfd = MAX(maxfd, fd);

  • branches/samba-3.3.x/source/smbd/server.c

    r423 r578  
    210210        /* We will abort gracefully when the client or remote system
    211211           goes away */
    212         smbd_set_server_fd(dup(0));
     212        int fd = dup(0);
     213
     214        if (fd < 0 || fd >= FD_SETSIZE) {
     215                return false;
     216        }
     217
     218        smbd_set_server_fd(fd);
    213219       
    214220        /* close our standard file descriptors */
    … …  
    437443                                                        ifss,
    438444                                                        true);
    439                                 if(s == -1) {
     445                                if(s < 0 || s >= FD_SETSIZE) {
     446                                        close(s);
    440447                                        continue;
    441448                                }
    … …  
    517524                                                &ss,
    518525                                                true);
    519                                 if (s == -1) {
     526                                if (s < 0 || s >= FD_SETSIZE) {
    520527                                        continue;
    521528                                }
    … …  
    710717                        socklen_t in_addrlen = sizeof(addr);
    711718                        pid_t child = 0;
     719                        int fd;
    712720
    713721                        s = -1;
    … …  
    722730                        }
    723731
    724                         smbd_set_server_fd(accept(s,&addr,&in_addrlen));
    725 
    726                         if (smbd_server_fd() == -1 && errno == EINTR)
     732                        fd = accept(s,&addr,&in_addrlen);
     733                        if (fd == -1 && errno == EINTR)
    727734                                continue;
    728 
    729                         if (smbd_server_fd() == -1) {
     735                        if (fd == -1) {
    730736                                DEBUG(2,("open_sockets_smbd: accept: %s\n",
    731737                                         strerror(errno)));
    732738                                continue;
    733739                        }
     740                        if (fd < 0 || fd >= FD_SETSIZE) {
     741                                DEBUG(2,("open_sockets_smbd: bad fd %d\n",
     742                                        fd ));
     743                                continue;
     744                        }
     745
     746                        smbd_set_server_fd(fd);
    734747
    735748                        /* Ensure child is set to blocking mode */

  • branches/samba-3.3.x/source/utils/smbfilter.c

    r206 r578  
    163163               
    164164                FD_ZERO(&fds);
    165                 if (s != -1) FD_SET(s, &fds);
    166                 if (c != -1) FD_SET(c, &fds);
     165                if (s >= 0 && s < FD_SETSIZE) FD_SET(s, &fds);
     166                if (c >= 0 && c < FD_SETSIZE) FD_SET(c, &fds);
    167167
    168168                num = sys_select_intr(MAX(s+1, c+1),&fds,NULL,NULL,NULL);
    … …  
    236236                socklen_t in_addrlen = sizeof(ss);
    237237               
     238                if (s < 0 || s >= FD_SETSIZE) {
     239                        break;
     240                }
     241
    238242                FD_ZERO(&fds);
    239243                FD_SET(s, &fds);

  • branches/samba-3.3.x/source/winbindd/winbindd.c

    r342 r578  
    837837        listen_priv_sock = open_winbindd_priv_socket();
    838838
    839         if (listen_sock == -1 || listen_priv_sock == -1) {
     839        if (listen_sock < 0 || listen_sock >= FD_SETSIZE ||
     840                        listen_priv_sock < 0 || listen_priv_sock >= FD_SETSIZE) {
    840841                perror("open_winbind_socket");
    841842                exit(1);
    … …  
    862863        FD_ZERO(&r_fds);
    863864        FD_ZERO(&w_fds);
     865
     866        /* We check the range for listen_sock and
     867           listen_priv_sock above. */
    864868        FD_SET(listen_sock, &r_fds);
    865869        FD_SET(listen_priv_sock, &r_fds);
    … …  
    891895
    892896        for (ev = fd_events; ev; ev = ev->next) {
     897                if (ev->fd < 0 || ev->fd >= FD_SETSIZE) {
     898                        /* Ignore here - event_add_to_select_args
     899                           should make this impossible. */
     900                        continue;
     901                }
     902
    893903                if (ev->flags & EVENT_FD_READ) {
    894904                        FD_SET(ev->fd, &r_fds);

  • branches/samba-3.3.x/source/winbindd/winbindd_dual.c

    r221 r578  
    12511251        }
    12521252
     1253        if (fdpair[0] < 0 || fdpair[0] >= FD_SETSIZE) {
     1254                DEBUG(0, ("fork_domain_child: bad fd range (%d)\n", fdpair[0]));
     1255                errno = EBADF;
     1256                return False;
     1257        }
     1258
    12531259        ZERO_STRUCT(state);
    12541260        state.pid = sys_getpid();
    … …  
    14061412
    14071413                FD_ZERO(&read_fds);
     1414                /* We check state.sock against FD_SETSIZE above. */
    14081415                FD_SET(state.sock, &read_fds);
    14091416
Note: See TracChangeset for help on using the changeset viewer.