Làm thế nào để giải quyết cảnh báo C4191 xung quanh các cuộc gọi đến GetProcAddress với FARPROC?

Question

Gần đây tôi cố gắng sử dụng /Wall Visual C++ option để cho phép tất cả các cảnh báo và phát hiện ra rằng đoạn mã sau:

typedef BOOL (WINAPI * TIsWow64ProcessFunction)(HANDLE, BOOL*); 
TIsWow64ProcessFunction isWow64ProcessFunction = reinterpret_cast<TIsWow64ProcessFunction> (
    ::GetProcAddress(kernel32DllHandle, "IsWow64Process")); 

sinh ra C4191:

warning C4191: 'reinterpret_cast' : unsafe conversion from 'FARPROC' to 'TIsWow64ProcessFunction' 
Calling this function through the result pointer may cause your program to fail 

Nếu tôi sử dụng một C-style cast cảnh báo tương tự xuất hiện nhưng bây giờ nó đề cập đến "type cast" thay vì "reinterpret_cast".

Cảnh báo tương tự được lặp lại cho mọi trường hợp tôi gọi GetProcAddress() và chuyển giá trị trả về của nó thành một số con trỏ hàm có thể sử dụng.

Làm cách nào để giải quyết các cảnh báo này? Tôi có cần thay đổi mã của mình không?

Answer 1

Bạn đang truyền một hàm FARPROC (con trỏ hàm không có arg) đến một con trỏ hàm với arg. Thông thường đây là một điều cực kỳ câm để làm điều đó có thể sẽ dẫn đến sự tham nhũng ngăn xếp. Bây giờ nó chỉ ra rằng GetProcAddress() không thực sự trả về một FARPROC và bạn thực sự biết những gì bạn đang làm - nhưng trình biên dịch không biết điều đó và nó cảm thấy bắt buộc phải cảnh báo bạn.

Cách duy nhất bạn có thể tắt tiếng là sử dụng khóa #pragma hoặc trình biên dịch để tắt cảnh báo. Nó là xấu xí và lộn xộn, nhưng đó là lập trình Windows cho bạn. :-)

Answer 2

Về cơ bản trình biên dịch không thể đảm bảo rằng hàm này thuộc loại thích hợp, vì vậy không an toàn để bạn gọi con trỏ kết quả. Tuy nhiên, trong một chương trình VS bạn không phải liên kết hoặc tải lên Windows .dll một cách rõ ràng, chúng sẽ được tải cho bạn và bất kỳ chức năng nào trong tiêu đề Windows luôn sẵn sàng để sử dụng.

Answer 3

Như câu trả lời khác đã đề cập, đây là một cảnh báo hữu ích. Normally, this type of coercion would be a serious bug hiding in your application.

Vì vậy, có thể bạn không muốn vô hiệu hóa nó trên toàn cầu bằng công cụ chuyển đổi trình biên dịch. Tuy nhiên, bạn vẫn cần phải gọi GetProcAddress và bạn thích các bản dựng của mình để biên dịch một cách rõ ràng mà không cần cảnh báo.

Bạn có hai lựa chọn tốt:

1. Suppress từng cảnh báo cá nhân sử dụng một pragma MSVC cụ thể. Trong một dòng mới ngay trên dàn diễn viên bẩn, thêm đoạn mã sau:

   #pragma warning(suppress: 4191) 
   

   này sẽ bỏ qua cảnh báo cho các dòng rất cạnh mã chỉ, đảm bảo rằng nó không phải là đàn áp trên toàn cầu và bạn sẽ vẫn nhận được cảnh báo nếu bạn cố gắng làm điều gì đó ngu ngốc ở nơi khác trong cơ sở mã. Tất nhiên, bạn sẽ cần phải thêm điều này mỗi khi bạn sử dụng GetProcAddress, đó là một loại đau. Tệ hơn nữa, đó là một tiện ích mở rộng không phải MSVC dành riêng cho di động làm xấu mã của bạn.

   Vì vậy, thay thế & hellip;
   
   

2. Bạn có thể tắt các cảnh báo bằng đúc một cách rõ ràng là kết quả của GetProcAddress (một FARPROC) để void*, và sau đó đúc mà void* để loại hàm con trỏ cụ thể. Ví dụ:

   typedef BOOL (__stdcall *TIsWow64ProcessFunction)(HANDLE, BOOL*); 
   
   TIsWow64ProcessFunction isWow64ProcessFunction = 
       reinterpret_cast<TIsWow64ProcessFunction>(
        reinterpret_cast<void*>(
        ::GetProcAddress(hInstance, "IsWow64Process"))); 
   

   Cách tiếp cận này sẽ hoạt động với các trình biên dịch khác, hơi kém xấu xí và được cho là mang ý nghĩa ngữ nghĩa hơn.