git: e4de0a965faa - stable/14 - hptrr(4): Stop checking for failures from malloc(M_WAITOK)

[Zhenlei Huang]

The branch stable/14 has been updated by zlei:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=e4de0a965faaff171e6061714478c8f4eaec8cb1

commit e4de0a965faaff171e6061714478c8f4eaec8cb1
Author:     Zhenlei Huang <z...@freebsd.org>
AuthorDate: 2024-09-03 10:25:29 +0000
Commit:     Zhenlei Huang <z...@freebsd.org>
CommitDate: 2024-09-30 04:44:23 +0000

    hptrr(4): Stop checking for failures from malloc(M_WAITOK)
    
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D45852
    
    (cherry picked from commit 92b0370ec65d5287a1deec84fd513e320a8da736)
---
 sys/dev/hptrr/hptrr_osm_bsd.c | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/sys/dev/hptrr/hptrr_osm_bsd.c b/sys/dev/hptrr/hptrr_osm_bsd.c
index 4208dd620e37..055ad13b14cb 100644
--- a/sys/dev/hptrr/hptrr_osm_bsd.c
+++ b/sys/dev/hptrr/hptrr_osm_bsd.c
@@ -1032,10 +1032,6 @@ static void hpt_final_init(void *dummy)
 
                for (i=0; i<os_max_queue_comm; i++) {
                        POS_CMDEXT ext = (POS_CMDEXT)malloc(sizeof(OS_CMDEXT), 
M_DEVBUF, M_WAITOK);
-                       if (!ext) {
-                               os_printk("Can't alloc cmdext(%d)", i);
-                               return ;
-                       }
                        ext->vbus_ext = vbus_ext;
                        ext->next = vbus_ext->cmdext_list;
                        vbus_ext->cmdext_list = ext;
@@ -1252,19 +1248,14 @@ static int hpt_ioctl(struct cdev *dev, u_long cmd, 
caddr_t data, int fflag, stru
 
                if (ioctl_args.nInBufferSize) {
                        ioctl_args.lpInBuffer = 
malloc(ioctl_args.nInBufferSize, M_DEVBUF, M_WAITOK);
-                       if (!ioctl_args.lpInBuffer)
-                               goto invalid;
                        if (copyin((void*)piop->lpInBuffer,
                                        ioctl_args.lpInBuffer, 
piop->nInBufferSize))
                                goto invalid;
                }
        
-               if (ioctl_args.nOutBufferSize) {
+               if (ioctl_args.nOutBufferSize)
                        ioctl_args.lpOutBuffer = 
malloc(ioctl_args.nOutBufferSize, M_DEVBUF, M_WAITOK | M_ZERO);
-                       if (!ioctl_args.lpOutBuffer)
-                               goto invalid;
-               }
-               
+
                hpt_do_ioctl(&ioctl_args);
        
                if (ioctl_args.result==HPT_IOCTL_RESULT_OK) {