Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
On Tue, Feb 4, 2014 at 5:51 PM, Dan Carpenter wrote: > On Tue, Feb 04, 2014 at 05:26:57PM +0900, SeongJae Park wrote: >> Hello, >> >> On Tue, Feb 4, 2014 at 5:21 PM, Dan Carpenter >> wrote: >> > On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: >> >> Fix the pointer-integer size mismatch warnings below: >> >> drivers/staging/bcm/CmHost.c: In function >> >> ‘StoreCmControlResponseMessage’: >> >> drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from >> >> integer of different size [-Wint-to-pointer-cast] >> >> pstAddIndication->psfAuthorizedSet = (struct >> >> bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfAuthorizedSet); >> >> ^ >> >> drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from >> >> integer of different size [-Wint-to-pointer-cast] >> >> pstAddIndication->psfAdmittedSet = (struct bcm_connect_mgr_params >> >> *)ntohl((ULONG)pstAddIndication->psfAdmittedSet); >> >>^ >> >> drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from >> >> integer of different size [-Wint-to-pointer-cast] >> >> pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params >> >> *)ntohl((ULONG)pstAddIndication->psfActiveSet); >> >> ^ >> > >> > No. You haven't fixed the bug, you've just silenced the warning so now >> > someone has to spent hours and hours looking for it when it comes time >> > to write a proper fix. Don't do that. >> > >> >> I had thought silencing warning may be better in someway for some people. >> But, anyway, now I agree and respect your opinion. Thank you for the opinion >> :) > > Don't worry about it. You're not the first person to try submit that > patch. > > When I last looked at it, this driver only worked under 32 bits and > there were other 64 bit bugs as well besides this one. > > regards, > dan carpenter > Thank you very much for your kind reply. I'm yet newbie to kernel community and thinking I should try with unavoidable mistakes. Your reply have gave me more courage for trial. I will not forget the learning I got from this trial :) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
On Tue, Feb 04, 2014 at 05:26:57PM +0900, SeongJae Park wrote: > Hello, > > On Tue, Feb 4, 2014 at 5:21 PM, Dan Carpenter > wrote: > > On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: > >> Fix the pointer-integer size mismatch warnings below: > >> drivers/staging/bcm/CmHost.c: In function > >> ‘StoreCmControlResponseMessage’: > >> drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from > >> integer of different size [-Wint-to-pointer-cast] > >> pstAddIndication->psfAuthorizedSet = (struct > >> bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfAuthorizedSet); > >> ^ > >> drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from > >> integer of different size [-Wint-to-pointer-cast] > >> pstAddIndication->psfAdmittedSet = (struct bcm_connect_mgr_params > >> *)ntohl((ULONG)pstAddIndication->psfAdmittedSet); > >>^ > >> drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from > >> integer of different size [-Wint-to-pointer-cast] > >> pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params > >> *)ntohl((ULONG)pstAddIndication->psfActiveSet); > >> ^ > > > > No. You haven't fixed the bug, you've just silenced the warning so now > > someone has to spent hours and hours looking for it when it comes time > > to write a proper fix. Don't do that. > > > > I had thought silencing warning may be better in someway for some people. > But, anyway, now I agree and respect your opinion. Thank you for the opinion > :) Don't worry about it. You're not the first person to try submit that patch. When I last looked at it, this driver only worked under 32 bits and there were other 64 bit bugs as well besides this one. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
Hello, On Tue, Feb 4, 2014 at 5:21 PM, Dan Carpenter wrote: > On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: >> Fix the pointer-integer size mismatch warnings below: >> drivers/staging/bcm/CmHost.c: In function >> ‘StoreCmControlResponseMessage’: >> drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from >> integer of different size [-Wint-to-pointer-cast] >> pstAddIndication->psfAuthorizedSet = (struct bcm_connect_mgr_params >> *)ntohl((ULONG)pstAddIndication->psfAuthorizedSet); >> ^ >> drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from >> integer of different size [-Wint-to-pointer-cast] >> pstAddIndication->psfAdmittedSet = (struct bcm_connect_mgr_params >> *)ntohl((ULONG)pstAddIndication->psfAdmittedSet); >>^ >> drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from >> integer of different size [-Wint-to-pointer-cast] >> pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params >> *)ntohl((ULONG)pstAddIndication->psfActiveSet); >> ^ > > No. You haven't fixed the bug, you've just silenced the warning so now > someone has to spent hours and hours looking for it when it comes time > to write a proper fix. Don't do that. > I had thought silencing warning may be better in someway for some people. But, anyway, now I agree and respect your opinion. Thank you for the opinion :) > regards, > dan carpenter > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: > Fix the pointer-integer size mismatch warnings below: > drivers/staging/bcm/CmHost.c: In function > ‘StoreCmControlResponseMessage’: > drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from > integer of different size [-Wint-to-pointer-cast] > pstAddIndication->psfAuthorizedSet = (struct bcm_connect_mgr_params > *)ntohl((ULONG)pstAddIndication->psfAuthorizedSet); > ^ > drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from > integer of different size [-Wint-to-pointer-cast] > pstAddIndication->psfAdmittedSet = (struct bcm_connect_mgr_params > *)ntohl((ULONG)pstAddIndication->psfAdmittedSet); >^ > drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from > integer of different size [-Wint-to-pointer-cast] > pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params > *)ntohl((ULONG)pstAddIndication->psfActiveSet); > ^ No. You haven't fixed the bug, you've just silenced the warning so now someone has to spent hours and hours looking for it when it comes time to write a proper fix. Don't do that. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: Fix the pointer-integer size mismatch warnings below: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAuthorizedSet); ^ drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAdmittedSet); ^ drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfActiveSet); ^ No. You haven't fixed the bug, you've just silenced the warning so now someone has to spent hours and hours looking for it when it comes time to write a proper fix. Don't do that. regards, dan carpenter -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
Hello, On Tue, Feb 4, 2014 at 5:21 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: Fix the pointer-integer size mismatch warnings below: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAuthorizedSet); ^ drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAdmittedSet); ^ drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfActiveSet); ^ No. You haven't fixed the bug, you've just silenced the warning so now someone has to spent hours and hours looking for it when it comes time to write a proper fix. Don't do that. I had thought silencing warning may be better in someway for some people. But, anyway, now I agree and respect your opinion. Thank you for the opinion :) regards, dan carpenter -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
On Tue, Feb 04, 2014 at 05:26:57PM +0900, SeongJae Park wrote: Hello, On Tue, Feb 4, 2014 at 5:21 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: Fix the pointer-integer size mismatch warnings below: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAuthorizedSet); ^ drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAdmittedSet); ^ drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfActiveSet); ^ No. You haven't fixed the bug, you've just silenced the warning so now someone has to spent hours and hours looking for it when it comes time to write a proper fix. Don't do that. I had thought silencing warning may be better in someway for some people. But, anyway, now I agree and respect your opinion. Thank you for the opinion :) Don't worry about it. You're not the first person to try submit that patch. When I last looked at it, this driver only worked under 32 bits and there were other 64 bit bugs as well besides this one. regards, dan carpenter -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] staging: bcm: fix pointer-integer size mismatch warnings
On Tue, Feb 4, 2014 at 5:51 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Tue, Feb 04, 2014 at 05:26:57PM +0900, SeongJae Park wrote: Hello, On Tue, Feb 4, 2014 at 5:21 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Tue, Feb 04, 2014 at 03:59:23PM +0900, SeongJae Park wrote: Fix the pointer-integer size mismatch warnings below: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAuthorizedSet); ^ drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAdmittedSet); ^ drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfActiveSet); ^ No. You haven't fixed the bug, you've just silenced the warning so now someone has to spent hours and hours looking for it when it comes time to write a proper fix. Don't do that. I had thought silencing warning may be better in someway for some people. But, anyway, now I agree and respect your opinion. Thank you for the opinion :) Don't worry about it. You're not the first person to try submit that patch. When I last looked at it, this driver only worked under 32 bits and there were other 64 bit bugs as well besides this one. regards, dan carpenter Thank you very much for your kind reply. I'm yet newbie to kernel community and thinking I should try with unavoidable mistakes. Your reply have gave me more courage for trial. I will not forget the learning I got from this trial :) -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] staging: bcm: fix pointer-integer size mismatch warnings
Fix the pointer-integer size mismatch warnings below: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication->psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfAuthorizedSet); ^ drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication->psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfAdmittedSet); ^ drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfActiveSet); ^ Signed-off-by: SeongJae Park --- drivers/staging/bcm/CmHost.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/bcm/CmHost.c b/drivers/staging/bcm/CmHost.c index cc91b5e..dd8f8f7 100644 --- a/drivers/staging/bcm/CmHost.c +++ b/drivers/staging/bcm/CmHost.c @@ -1384,7 +1384,8 @@ ULONG StoreCmControlResponseMessage(struct bcm_mini_adapter *Adapter, PVOID pvBu } /* this can't possibly be right */ - pstAddIndication->psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfAuthorizedSet); + pstAddIndication->psfAuthorizedSet = (struct bcm_connect_mgr_params *) + (uintptr_t)ntohl((ULONG)pstAddIndication->psfAuthorizedSet); if (pstAddIndicationAlt->u8Type == DSA_REQ) { struct bcm_add_request AddRequest; @@ -1423,7 +1424,8 @@ ULONG StoreCmControlResponseMessage(struct bcm_mini_adapter *Adapter, PVOID pvBu return 0; } - pstAddIndication->psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfAdmittedSet); + pstAddIndication->psfAdmittedSet = (struct bcm_connect_mgr_params *) + (uintptr_t)ntohl((ULONG)pstAddIndication->psfAdmittedSet); /* ACTIVE SET */ pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params *) @@ -1437,7 +1439,8 @@ ULONG StoreCmControlResponseMessage(struct bcm_mini_adapter *Adapter, PVOID pvBu return 0; } - pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication->psfActiveSet); + pstAddIndication->psfActiveSet = (struct bcm_connect_mgr_params *) + (uintptr_t)ntohl((ULONG)pstAddIndication->psfActiveSet); (*puBufferLength) = sizeof(struct bcm_add_indication); *(struct bcm_add_indication *)pvBuffer = *pstAddIndication; -- 1.8.3.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] staging: bcm: fix pointer-integer size mismatch warnings
Fix the pointer-integer size mismatch warnings below: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1387:39: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAuthorizedSet); ^ drivers/staging/bcm/CmHost.c:1426:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAdmittedSet); ^ drivers/staging/bcm/CmHost.c:1440:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfActiveSet); ^ Signed-off-by: SeongJae Park sj38.p...@gmail.com --- drivers/staging/bcm/CmHost.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/bcm/CmHost.c b/drivers/staging/bcm/CmHost.c index cc91b5e..dd8f8f7 100644 --- a/drivers/staging/bcm/CmHost.c +++ b/drivers/staging/bcm/CmHost.c @@ -1384,7 +1384,8 @@ ULONG StoreCmControlResponseMessage(struct bcm_mini_adapter *Adapter, PVOID pvBu } /* this can't possibly be right */ - pstAddIndication-psfAuthorizedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAuthorizedSet); + pstAddIndication-psfAuthorizedSet = (struct bcm_connect_mgr_params *) + (uintptr_t)ntohl((ULONG)pstAddIndication-psfAuthorizedSet); if (pstAddIndicationAlt-u8Type == DSA_REQ) { struct bcm_add_request AddRequest; @@ -1423,7 +1424,8 @@ ULONG StoreCmControlResponseMessage(struct bcm_mini_adapter *Adapter, PVOID pvBu return 0; } - pstAddIndication-psfAdmittedSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfAdmittedSet); + pstAddIndication-psfAdmittedSet = (struct bcm_connect_mgr_params *) + (uintptr_t)ntohl((ULONG)pstAddIndication-psfAdmittedSet); /* ACTIVE SET */ pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *) @@ -1437,7 +1439,8 @@ ULONG StoreCmControlResponseMessage(struct bcm_mini_adapter *Adapter, PVOID pvBu return 0; } - pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *)ntohl((ULONG)pstAddIndication-psfActiveSet); + pstAddIndication-psfActiveSet = (struct bcm_connect_mgr_params *) + (uintptr_t)ntohl((ULONG)pstAddIndication-psfActiveSet); (*puBufferLength) = sizeof(struct bcm_add_indication); *(struct bcm_add_indication *)pvBuffer = *pstAddIndication; -- 1.8.3.2 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/