fix #4286,fifx memory leak when sal_socket failed

[jianbaoshan]

拉取/合并请求描述：(PR description)

问题描述

发现个内存泄漏的地方，解决特殊情况下出现的内存泄漏问题，将申请的内存直接进行初始化，而不是在其他条件下执行成功才初始化，执行失败不进行初始化直接释放内存，这段内存只有正确初始化之后才会被释放掉，否则不会被释放掉

修改方案

将申请的内存直接进行初始化，而不是在其他条件下执行成功才初始化，执行失败不进行初始化直接释放内存

测试验证

• BSP:rt-thread\bsp\hc32\ev_hc32f4a0_lqfp176

• .config:rt-thread默认配置UDP数量为4

• action:开启超过4个的udp连接，此时sal_socket会执行失败，失败原因是UDP数量不够，失败的时候不会执行dfs_vnode_init初始化函数对申请的vnode内存进行初始化，直接就释放该内存，但是释放该内存的依据是ref_count--为0，未进行初始化的时候，ret_count是个随机值，进行自减极大概率不会变成0，导致这块内存实际上并没有被释放掉

当前拉取/合并请求的状态 Intent for your PR

必须选择一项 Choose one (Mandatory):

• 本拉取/合并请求是一个草稿版本 This PR is for a code-review and is intended to get feedback
• 本拉取/合并请求是一个成熟版本 This PR is mature, and ready to be integrated into the repo

代码质量 Code Quality：

我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:

• 已经仔细查看过代码改动的对比 Already check the difference between PR and old code
• 代码风格正确，包括缩进空格，命名及其他风格 Style guide is adhered to, including spacing, naming and other styles
• 没有垃圾代码，代码尽量精简，不包含#if 0代码，不包含已经被注释了的代码 All redundant code is removed and cleaned up
• 所有变更均有原因及合理的，并且不会影响到其他软件组件代码或BSP All modifications are justified and not affect other components or BSP
• 对难懂代码均提供对应的注释 I've commented appropriately where code is tricky
• 代码是高质量的 Code in this PR is of high quality
• 已经使用formatting 等源码格式化工具确保格式符合RT-Thread代码规范 This PR complies with RT-Thread code specification
• 如果是新增bsp, 已经添加ci检查到.github/workflows/bsp_buildings.yml 详细请参考链接BSP自查

[CLAassistant]

All committers have signed the CLA.

[github-actions]

📌 Code Review Assignment

🏷️ Tag: components

Path: components
Reviewers: @Maihuanyi

Changed Files (Click to expand)

• components/net/sal/socket/net_sockets.c

---

📊 Current Review Status (Last Updated: 2025-07-14 08:34 UTC)

• ⌛ @Maihuanyi Pending Review

---

📝 Review Instructions

1. 维护者可以通过单击此处来刷新审查状态: 🔄 刷新状态
   Maintainers can refresh the review status by clicking here: 🔄 Refresh Status

2. 确认审核通过后评论 LGTM/lgtm
   Comment LGTM/lgtm after confirming approval

3. PR合并前需至少一位维护者确认
   PR must be confirmed by at least one maintainer before merging

ℹ️ 刷新CI状态操作需要具备仓库写入权限。
ℹ️ Refresh CI status operation requires repository Write permission.

[jianbaoshan]

@BernardXiong 请帮忙评审 谢谢

[Ryan-CW-Code]

很不错，这种隐藏bug是比较难发现的。
或许更推荐的操作是使用dfs_vnode_create替代malloc
下图中都是用malloc创建vnode的地方，虽然他们都进行正确的处理了，但是统一使用dfs_vnode_create会更规范些

@Rbb666 @kurisaW @Maihuanyi 大佬们可以评估一下

[Maihuanyi]

你好，请签一下CLA @jianbaoshan

[jianbaoshan]

已经签了

[Maihuanyi]

LGTM

[Rbb666]

很不错，这种隐藏bug是比较难发现的。 或许更推荐的操作是使用dfs_vnode_create替代malloc 下图中都是用malloc创建vnode的地方，虽然他们都进行正确的处理了，但是统一使用dfs_vnode_create会更规范些

@Rbb666 @kurisaW @Maihuanyi 大佬们可以评估一下

赞同，这样修改更合理