Ticket #12020: 0007-Fix-GCC-5-maybe-uninitialized-warnings.patch

File 0007-Fix-GCC-5-maybe-uninitialized-warnings.patch, 8.4 KB (added by mt, 9 years ago)

  • src/add-ons/accelerants/radeon_hd/atombios/atom.cpp 

    From 98f55991d985c0252f4777b3bdfc6fd866db7043 Mon Sep 17 00:00:00 2001
From: Murai Takashi <tmurai01@gmail.com>
Date: Sat, 25 Apr 2015 08:08:00 +0900
Subject: [PATCH 7/7] Fix GCC 5 maybe-uninitialized warnings.

---
 .../accelerants/radeon_hd/atombios/atom.cpp        | 22 +++++++++++-----------
 src/add-ons/kernel/file_systems/bfs/BPlusTree.cpp  |  6 +++---
 .../x86_physical_page_mapper_large_memory.cpp      |  2 +-
 .../kernel/device_manager/device_manager.cpp       |  2 +-
 src/system/kernel/fs/vfs.cpp                       |  2 +-
 src/system/kernel/messaging/MessagingService.cpp   |  2 +-
 src/system/libroot/os/driver_settings.cpp          |  2 +-
 7 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/src/add-ons/accelerants/radeon_hd/atombios/atom.cpp b/src/add-ons/accelerants/radeon_hd/atombios/atom.cpp
index 6c61e53..2afcd26 100644
    a b static void  
    496496atom_op_add(atom_exec_context *ctx, int *ptr, int arg)
    497497{
    498498    uint8 attr = U8((*ptr)++);
    499     uint32 dst, src, saved;
     499    uint32 dst, src, saved = 0;
    500500    int dptr = *ptr;
    501501    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    502502    src = atom_get_src(ctx, attr, ptr);
    … … static void  
    513513atom_op_and(atom_exec_context *ctx, int *ptr, int arg)
    514514{
    515515    uint8 attr = U8((*ptr)++);
    516     uint32 dst, src, saved;
     516    uint32 dst, src, saved = 0;
    517517    int dptr = *ptr;
    518518    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    519519    src = atom_get_src(ctx, attr, ptr);
    … … static void  
    559559atom_op_clear(atom_exec_context *ctx, int *ptr, int arg)
    560560{
    561561    uint8 attr = U8((*ptr)++);
    562     uint32 saved;
     562    uint32 saved = 0;
    563563    int dptr = *ptr;
    564564    attr &= 0x38;
    565565    attr |= atom_def_dst[attr>>3]<<6;
    … … static void  
    684684atom_op_mask(atom_exec_context *ctx, int *ptr, int arg)
    685685{
    686686    uint8 attr = U8((*ptr)++);
    687     uint32 dst, mask, src, saved;
     687    uint32 dst, mask, src, saved = 0;
    688688    int dptr = *ptr;
    689689    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    690690    mask = atom_get_src_direct(ctx, ((attr >> 3) & 7), ptr);
    … … static void  
    740740atom_op_or(atom_exec_context *ctx, int *ptr, int arg)
    741741{
    742742    uint8 attr = U8((*ptr)++);
    743     uint32 dst, src, saved;
     743    uint32 dst, src, saved = 0;
    744744    int dptr = *ptr;
    745745    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    746746    src = atom_get_src(ctx, attr, ptr);
    … … atom_op_setregblock(atom_exec_context *ctx, int *ptr, int arg)  
    848848static void atom_op_shift_left(atom_exec_context *ctx, int *ptr, int arg)
    849849{
    850850    uint8 attr = U8((*ptr)++), shift;
    851     uint32 saved, dst;
     851    uint32 saved = 0, dst;
    852852    int dptr = *ptr;
    853853    attr &= 0x38;
    854854    attr |= atom_def_dst[attr >> 3] << 6;
    … … static void atom_op_shift_left(atom_exec_context *ctx, int *ptr, int arg)  
    866866static void atom_op_shift_right(atom_exec_context *ctx, int *ptr, int arg)
    867867{
    868868    uint8 attr = U8((*ptr)++), shift;
    869     uint32 saved, dst;
     869    uint32 saved = 0, dst;
    870870    int dptr = *ptr;
    871871    attr &= 0x38;
    872872    attr |= atom_def_dst[attr >> 3] << 6;
    … … static void atom_op_shift_right(atom_exec_context *ctx, int *ptr, int arg)  
    884884static void atom_op_shl(atom_exec_context *ctx, int *ptr, int arg)
    885885{
    886886    uint8 attr = U8((*ptr)++), shift;
    887     uint32 saved, dst;
     887    uint32 saved = 0, dst;
    888888    int dptr = *ptr;
    889889    uint32 dst_align = atom_dst_to_src[(attr >> 3) & 7][(attr >> 6) & 3];
    890890    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    … … static void atom_op_shl(atom_exec_context *ctx, int *ptr, int arg)  
    905905static void atom_op_shr(atom_exec_context *ctx, int *ptr, int arg)
    906906{
    907907    uint8 attr = U8((*ptr)++), shift;
    908     uint32 saved, dst;
     908    uint32 saved = 0, dst;
    909909    int dptr = *ptr;
    910910    uint32 dst_align = atom_dst_to_src[(attr >> 3) & 7][(attr >> 6) & 3];
    911911    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    … … static void  
    927927atom_op_sub(atom_exec_context *ctx, int *ptr, int arg)
    928928{
    929929    uint8 attr = U8((*ptr)++);
    930     uint32 dst, src, saved;
     930    uint32 dst, src, saved = 0;
    931931    int dptr = *ptr;
    932932    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    933933    src = atom_get_src(ctx, attr, ptr);
    … … static void  
    984984atom_op_xor(atom_exec_context *ctx, int *ptr, int arg)
    985985{
    986986    uint8 attr = U8((*ptr)++);
    987     uint32 dst, src, saved;
     987    uint32 dst, src, saved = 0;
    988988    int dptr = *ptr;
    989989    dst = atom_get_dst(ctx, arg, attr, ptr, &saved, 1);
    990990    src = atom_get_src(ctx, attr, ptr);

  • src/add-ons/kernel/file_systems/bfs/BPlusTree.cpp 

    diff --git a/src/add-ons/kernel/file_systems/bfs/BPlusTree.cpp b/src/add-ons/kernel/file_systems/bfs/BPlusTree.cpp
index 9a6cf4e..330ed6e 100644
    a b BPlusTree::_FindKey(const bplustree_node* node, const uint8* key,  
    10621062    for (int16 first = 0, last = node->NumKeys() - 1; first <= last;) {
    10631063        uint16 i = (first + last) >> 1;
    10641064
    1065         uint16 searchLength;
     1065        uint16 searchLength = 0;
    10661066        uint8* searchKey = node->KeyAt(i, &searchLength);
    10671067        if (searchKey + searchLength + sizeof(off_t) + sizeof(uint16)
    10681068                > (uint8*)node + fNodeSize
    … … TreeIterator::Traverse(int8 direction, void* key, uint16* keyLength,  
    27322732    if (node->all_key_count == 0)
    27332733        RETURN_ERROR(B_ERROR);  // B_ENTRY_NOT_FOUND ?
    27342734
    2735     uint16 length;
     2735    uint16 length = 0;
    27362736    uint8* keyStart = node->KeyAt(fCurrentKey, &length);
    27372737    if (keyStart + length + sizeof(off_t) + sizeof(uint16)
    27382738            > (uint8*)node + fTree->fNodeSize
    … … bplustree_node::CheckIntegrity(uint32 nodeSize) const  
    29912991        DEBUGGER(("invalid node: key/length count"));
    29922992
    29932993    for (int32 i = 0; i < NumKeys(); i++) {
    2994         uint16 length;
     2994        uint16 length = 0;
    29952995        uint8* key = KeyAt(i, &length);
    29962996        if (key + length + sizeof(off_t) + sizeof(uint16)
    29972997                > (uint8*)this + nodeSize

  • src/system/kernel/arch/x86/paging/x86_physical_page_mapper_large_memory.cpp 

    diff --git a/src/system/kernel/arch/x86/paging/x86_physical_page_mapper_large_memory.cpp b/src/system/kernel/arch/x86/paging/x86_physical_page_mapper_large_memory.cpp
index 07c9446..e816af1 100644
    a b status_t  
    492492LargeMemoryPhysicalPageMapper::GetPage(phys_addr_t physicalAddress,
    493493    addr_t* virtualAddress, void** handle)
    494494{
    495     PhysicalPageSlot* slot;
     495    PhysicalPageSlot* slot = NULL;
    496496    status_t error = GetSlot(true, slot);
    497497    if (error != B_OK)
    498498        return error;

  • src/system/kernel/device_manager/device_manager.cpp 

    diff --git a/src/system/kernel/device_manager/device_manager.cpp b/src/system/kernel/device_manager/device_manager.cpp
index 5537aa0..f42a030 100644
    a b init_node_tree(void)  
    22092209        {NULL}
    22102210    };
    22112211
    2212     device_node* node;
     2212    device_node* node = NULL;
    22132213    if (register_node(NULL, DEVICE_MANAGER_ROOT_NAME, attrs, NULL, &node)
    22142214            != B_OK) {
    22152215        dprintf("Cannot register Devices Root Node\n");

  • src/system/kernel/fs/vfs.cpp 

    diff --git a/src/system/kernel/fs/vfs.cpp b/src/system/kernel/fs/vfs.cpp
index 6e98ac4..1cd8d69 100644
    a b vfs_get_vnode_from_path(const char* path, bool kernel, struct vnode** _vnode)  
    41034103extern "C" status_t
    41044104vfs_get_vnode(dev_t mountID, ino_t vnodeID, bool canWait, struct vnode** _vnode)
    41054105{
    4106     struct vnode* vnode;
     4106    struct vnode* vnode = NULL;
    41074107
    41084108    status_t status = get_vnode(mountID, vnodeID, &vnode, canWait, false);
    41094109    if (status != B_OK)

  • src/system/kernel/messaging/MessagingService.cpp 

    diff --git a/src/system/kernel/messaging/MessagingService.cpp b/src/system/kernel/messaging/MessagingService.cpp
index 728a31e..4b59f25 100644
    a b _user_register_messaging_service(sem_id lockSem, sem_id counterSem)  
    568568    if (!sMessagingService->Lock())
    569569        return B_BAD_VALUE;
    570570
    571     area_id areaID;
     571    area_id areaID = 0;
    572572    status_t error = sMessagingService->RegisterService(lockSem, counterSem,
    573573        areaID);
    574574

  • src/system/libroot/os/driver_settings.cpp 

    diff --git a/src/system/libroot/os/driver_settings.cpp b/src/system/libroot/os/driver_settings.cpp
index 17a6473..9004095 100644
    a b parse_parameter(struct driver_parameter *parameter, char **_pos, int32 level)  
    282282    status = get_word(&pos, &parameter->name, NO_ASSIGNMENT, true);
    283283    if (status == CONTINUE_PARAMETER) {
    284284        while (status == CONTINUE_PARAMETER) {
    285             char **newArray, *value;
     285            char **newArray, *value = NULL;
    286286            status = get_word(&pos, &value, parameter->value_count == 0
    287287                ? ALLOW_ASSIGNMENT : IGNORE_ASSIGNMENT, false);
    288288            if (status < B_OK)