diff options
Diffstat (limited to 'drivers/acpi/executer/exconvrt.c')
-rw-r--r-- | drivers/acpi/executer/exconvrt.c | 27 |
1 files changed, 6 insertions, 21 deletions
diff --git a/drivers/acpi/executer/exconvrt.c b/drivers/acpi/executer/exconvrt.c index 04e5194989a6..e6f55cf8ad31 100644 --- a/drivers/acpi/executer/exconvrt.c +++ b/drivers/acpi/executer/exconvrt.c | |||
@@ -5,7 +5,7 @@ | |||
5 | *****************************************************************************/ | 5 | *****************************************************************************/ |
6 | 6 | ||
7 | /* | 7 | /* |
8 | * Copyright (C) 2000 - 2005, R. Byron Moore | 8 | * Copyright (C) 2000 - 2006, R. Byron Moore |
9 | * All rights reserved. | 9 | * All rights reserved. |
10 | * | 10 | * |
11 | * Redistribution and use in source and binary forms, with or without | 11 | * Redistribution and use in source and binary forms, with or without |
@@ -504,18 +504,12 @@ acpi_ex_convert_to_string(union acpi_operand_object * obj_desc, | |||
504 | } | 504 | } |
505 | 505 | ||
506 | /* | 506 | /* |
507 | * Perform the conversion. | 507 | * Create a new string object and string buffer |
508 | * (-1 because of extra separator included in string_length from above) | 508 | * (-1 because of extra separator included in string_length from above) |
509 | */ | 509 | */ |
510 | string_length--; | ||
511 | if (string_length > ACPI_MAX_STRING_CONVERSION) { /* ACPI limit */ | ||
512 | return_ACPI_STATUS(AE_AML_STRING_LIMIT); | ||
513 | } | ||
514 | |||
515 | /* Create a new string object and string buffer */ | ||
516 | |||
517 | return_desc = | 510 | return_desc = |
518 | acpi_ut_create_string_object((acpi_size) string_length); | 511 | acpi_ut_create_string_object((acpi_size) |
512 | (string_length - 1)); | ||
519 | if (!return_desc) { | 513 | if (!return_desc) { |
520 | return_ACPI_STATUS(AE_NO_MEMORY); | 514 | return_ACPI_STATUS(AE_NO_MEMORY); |
521 | } | 515 | } |
@@ -660,17 +654,8 @@ acpi_ex_convert_to_target_type(acpi_object_type destination_type, | |||
660 | break; | 654 | break; |
661 | 655 | ||
662 | default: | 656 | default: |
663 | ACPI_DEBUG_PRINT((ACPI_DB_ERROR, | 657 | ACPI_REPORT_ERROR(("Unknown Target type ID 0x%X aml_opcode %X dest_type %s\n", GET_CURRENT_ARG_TYPE(walk_state->op_info->runtime_args), walk_state->opcode, acpi_ut_get_type_name(destination_type))); |
664 | "Unknown Target type ID 0x%X Op %s dest_type %s\n", | 658 | status = AE_AML_INTERNAL; |
665 | GET_CURRENT_ARG_TYPE(walk_state->op_info-> | ||
666 | runtime_args), | ||
667 | walk_state->op_info->name, | ||
668 | acpi_ut_get_type_name(destination_type))); | ||
669 | |||
670 | ACPI_REPORT_ERROR(("Bad Target Type (ARGI): %X\n", | ||
671 | GET_CURRENT_ARG_TYPE(walk_state->op_info-> | ||
672 | runtime_args))) | ||
673 | status = AE_AML_INTERNAL; | ||
674 | } | 659 | } |
675 | 660 | ||
676 | /* | 661 | /* |