src/os/solaris/vm/os_solaris.cpp

changeset 6349
7d28f4e15b61
parent 6348
0e6af9b390af
child 6518
62c54fcc0a35
     1.1 --- a/src/os/solaris/vm/os_solaris.cpp	Fri Jan 24 09:28:47 2014 +0100
     1.2 +++ b/src/os/solaris/vm/os_solaris.cpp	Wed Mar 05 11:28:33 2014 -0800
     1.3 @@ -1,5 +1,5 @@
     1.4  /*
     1.5 - * Copyright (c) 1997, 2013, Oracle and/or its affiliates. All rights reserved.
     1.6 + * Copyright (c) 1997, 2014, Oracle and/or its affiliates. All rights reserved.
     1.7   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
     1.8   *
     1.9   * This code is free software; you can redistribute it and/or modify it
    1.10 @@ -2232,8 +2232,8 @@
    1.11          st->cr();
    1.12          status = true;
    1.13        }
    1.14 -      ::close(fd);
    1.15      }
    1.16 +    ::close(fd);
    1.17    }
    1.18    return status;
    1.19  }
    1.20 @@ -2257,13 +2257,18 @@
    1.21                            "ILL_ILLTRP", "ILL_PRVOPC", "ILL_PRVREG",
    1.22                            "ILL_COPROC", "ILL_BADSTK" };
    1.23  
    1.24 +const size_t ill_names_length = (sizeof(ill_names)/sizeof(char *));
    1.25 +
    1.26  const char *fpe_names[] = { "FPE0", "FPE_INTDIV", "FPE_INTOVF", "FPE_FLTDIV",
    1.27                            "FPE_FLTOVF", "FPE_FLTUND", "FPE_FLTRES",
    1.28                            "FPE_FLTINV", "FPE_FLTSUB" };
    1.29 +const size_t fpe_names_length = (sizeof(fpe_names)/sizeof(char *));
    1.30  
    1.31  const char *segv_names[] = { "SEGV0", "SEGV_MAPERR", "SEGV_ACCERR" };
    1.32 +const size_t segv_names_length = (sizeof(segv_names)/sizeof(char *));
    1.33  
    1.34  const char *bus_names[] = { "BUS0", "BUS_ADRALN", "BUS_ADRERR", "BUS_OBJERR" };
    1.35 +const size_t bus_names_length = (sizeof(bus_names)/sizeof(char *));
    1.36  
    1.37  void os::print_siginfo(outputStream* st, void* siginfo) {
    1.38    st->print("siginfo:");
    1.39 @@ -2282,19 +2287,23 @@
    1.40    assert(c > 0, "unexpected si_code");
    1.41    switch (si->si_signo) {
    1.42    case SIGILL:
    1.43 -    st->print(", si_code=%d (%s)", c, c > 8 ? "" : ill_names[c]);
    1.44 +    st->print(", si_code=%d (%s)", c,
    1.45 +      c >= ill_names_length ? "" : ill_names[c]);
    1.46      st->print(", si_addr=" PTR_FORMAT, si->si_addr);
    1.47      break;
    1.48    case SIGFPE:
    1.49 -    st->print(", si_code=%d (%s)", c, c > 9 ? "" : fpe_names[c]);
    1.50 +    st->print(", si_code=%d (%s)", c,
    1.51 +      c >= fpe_names_length ? "" : fpe_names[c]);
    1.52      st->print(", si_addr=" PTR_FORMAT, si->si_addr);
    1.53      break;
    1.54    case SIGSEGV:
    1.55 -    st->print(", si_code=%d (%s)", c, c > 2 ? "" : segv_names[c]);
    1.56 +    st->print(", si_code=%d (%s)", c,
    1.57 +      c >= segv_names_length ? "" : segv_names[c]);
    1.58      st->print(", si_addr=" PTR_FORMAT, si->si_addr);
    1.59      break;
    1.60    case SIGBUS:
    1.61 -    st->print(", si_code=%d (%s)", c, c > 3 ? "" : bus_names[c]);
    1.62 +    st->print(", si_code=%d (%s)", c,
    1.63 +      c >= bus_names_length ? "" : bus_names[c]);
    1.64      st->print(", si_addr=" PTR_FORMAT, si->si_addr);
    1.65      break;
    1.66    default:
    1.67 @@ -3011,7 +3020,7 @@
    1.68  char *os::scan_pages(char *start, char* end, page_info* page_expected, page_info* page_found) {
    1.69    const uint_t info_types[] = { MEMINFO_VLGRP, MEMINFO_VPAGESIZE };
    1.70    const size_t types = sizeof(info_types) / sizeof(info_types[0]);
    1.71 -  uint64_t addrs[MAX_MEMINFO_CNT], outdata[types * MAX_MEMINFO_CNT];
    1.72 +  uint64_t addrs[MAX_MEMINFO_CNT], outdata[types * MAX_MEMINFO_CNT + 1];
    1.73    uint_t validity[MAX_MEMINFO_CNT];
    1.74  
    1.75    size_t page_size = MAX2((size_t)os::vm_page_size(), page_expected->size);
    1.76 @@ -3050,7 +3059,7 @@
    1.77        }
    1.78      }
    1.79  
    1.80 -    if (i != addrs_count) {
    1.81 +    if (i < addrs_count) {
    1.82        if ((validity[i] & 2) != 0) {
    1.83          page_found->lgrp_id = outdata[types * i];
    1.84        } else {

mercurial