Changeset 102


Ignore:
Timestamp:
Oct 2, 2017, 12:45:31 PM (8 years ago)
Author:
bird
Message:

gcc7 warning fixes

Location:
trunk/kLdr
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/kLdr/kLdrModLX.c

    r96 r102  
    14871487                case RANGE:
    14881488                    KLDRMODLX_ASSERT(!"RANGE");
     1489                    /* Falls through. */
    14891490                default:
    14901491                    rc = KLDR_ERR_LX_BAD_PAGE_MAP;
    … …  
    24442445                    case NRRENT:
    24452446                        KLDRMODLX_ASSERT(!"NRRENT");
     2447                        /* Falls through. */
    24462448                    default:
    24472449                        iSelector = -1;

  • trunk/kLdr/kLdrModMachO.c

    r91 r102  
    750750                                KLDRMODMACHO_CHECK_RETURN(fOpenFlags & KLDRMOD_OPEN_FLAGS_FOR_INFO, \
    751751                                                          KLDR_ERR_MACHO_UNSUPPORTED_INIT_SECTION); \
     752                                /* Falls through. */ \
    752753                            case S_MOD_TERM_FUNC_POINTERS: \
    753754                                /** @todo this requires a query API or flag... (e.g. C++ destructors) */ \
    … …  
    853854                                        *pLinkAddress = pSect->addr; \
    854855                                } \
    855                                 /* fall thru */ \
     856                                /* Falls through. */ \
    856857                            case MH_EXECUTE: \
    857858                            case MH_DYLIB: \
    … …  
    18551856            /** @todo implement indirect and prebound symbols. */
    18561857        default:
    1857             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     1858            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    18581859    }
    18591860
    … …  
    19831984            /** @todo implement indirect and prebound symbols. */
    19841985        default:
    1985             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     1986            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    19861987    }
    19871988
    … …  
    20302031    }
    20312032    else
    2032         KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     2033        KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    20332034
    20342035    return rc;
    … …  
    21342135                /** @todo implement indirect and prebound symbols. */
    21352136            default:
    2136                 KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     2137                KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    21372138        }
    21382139
    … …  
    22462247                /** @todo implement indirect and prebound symbols. */
    22472248            default:
    2248                 KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     2249                KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    22492250        }
    22502251
    … …  
    27752776                                                   pModMachO->cSymbols, NewBaseAddress);
    27762777            else
    2777                 KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     2778                KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    27782779            if (rc)
    27792780                break;
    … …  
    28882889                    case MACHO_N_INDR:
    28892890                    case MACHO_N_PBUD:
    2890                         KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     2891                        KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    28912892                    default:
    2892                         KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_MACHO_BAD_SYMBOL);
     2893                        KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_MACHO_BAD_SYMBOL);
    28932894                }
    28942895            }
    … …  
    30543055            case 3: SymAddr = *uFixVirgin.pi64; break;
    30553056            default:
    3056                 KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_BAD_FIXUP);
     3057                KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_BAD_FIXUP);
    30573058        }
    30583059
    … …  
    30803081                        case MACHO_N_INDR:
    30813082                        case MACHO_N_PBUD:
    3082                             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     3083                            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    30833084                        default:
    3084                             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_MACHO_BAD_SYMBOL);
     3085                            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_MACHO_BAD_SYMBOL);
    30853086                    }
    30863087                    SymAddr = sizeof(KU64) * Fixup.r.r_symbolnum + pModMachO->GotRVA + NewBaseAddress;
    … …  
    30963097                case X86_64_RELOC_SIGNED_4:
    30973098                    KLDRMODMACHO_CHECK_RETURN(Fixup.r.r_pcrel, KLDR_ERR_BAD_FIXUP);
     3099                    /* Falls through. */
    30983100                default:
    30993101                {
    … …  
    31153117                            break;
    31163118                        default:
    3117                             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_BAD_FIXUP);
     3119                            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_BAD_FIXUP);
    31183120                    }
    31193121
    … …  
    31473149                        case MACHO_N_INDR:
    31483150                        case MACHO_N_PBUD:
    3149                             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     3151                            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    31503152                        default:
    3151                             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_MACHO_BAD_SYMBOL);
     3153                            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_MACHO_BAD_SYMBOL);
    31523154                    }
    31533155                    break;
    … …  
    31803182                        case MACHO_N_INDR:
    31813183                        case MACHO_N_PBUD:
    3182                             KLDRMODMACHO_CHECK_RETURN(0,KLDR_ERR_TODO);
     3184                            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    31833185                        default:
    3184                             KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_MACHO_BAD_SYMBOL);
     3186                            KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_MACHO_BAD_SYMBOL);
    31853187                    }
    31863188
    … …  
    32213223                            case MACHO_N_INDR:
    32223224                            case MACHO_N_PBUD:
    3223                                 KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     3225                                KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    32243226                            default:
    3225                                 KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_MACHO_BAD_SYMBOL);
     3227                                KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_MACHO_BAD_SYMBOL);
    32263228                        }
    32273229                    }
    … …  
    32343236                    }
    32353237                    else
    3236                         KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_BAD_FIXUP);
     3238                        KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_BAD_FIXUP);
    32373239                }
    32383240                break;
    … …  
    32613263                /*case X86_64_RELOC_SUBTRACTOR: - must be r_extern=1 says as. */
    32623264                default:
    3263                     KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_BAD_FIXUP);
     3265                    KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_BAD_FIXUP);
    32643266            }
    32653267            if (Fixup.r.r_symbolnum != R_ABS)
    … …  
    32943296                break;
    32953297            default:
    3296                 KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_BAD_FIXUP);
     3298                KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_BAD_FIXUP);
    32973299        }
    32983300    }
    … …  
    36723674
    36733675                default:
    3674                     KLDRMODMACHO_CHECK_RETURN(0, KLDR_ERR_TODO);
     3676                    KLDRMODMACHO_FAILED_RETURN(KLDR_ERR_TODO);
    36753677            }
    36763678        }
Note: See TracChangeset for help on using the changeset viewer.