From f59dd7e7572f998bfdc3abe381f4f186a3455042 Mon Sep 17 00:00:00 2001
From: Oracle Public Cloud User
 <opc@auranode.subnet09041427.vcn09041427.oraclevcn.com>
Date: Sat, 28 Mar 2026 14:21:09 +0000
Subject: [PATCH 1/2] CRITICAL FIX v39: FINAL umount -> sys_umount fix for
 kernel 4.9

The pre-patched core_hook.c.patched file contains umount() calls
that don't exist in kernel 4.9. This fix adds a v39 script that runs
AFTER the pre-patched file is applied to ensure ALL umount() calls
are replaced with sys_umount().

Issue: Build #53 failed with undefined reference to 'umount' at line 566
Root Cause: Pre-patched file overwrites v38 fixes
Solution: Run v39 fix after pre-patched file application

Changes:
- Added fix_kernelsu_49_v39.py with aggressive umount() replacement
- Updated workflow to run v39 after pre-patched file application
- Ensures sys_umount declaration is added for kernel 4.9
---
 .../workflows/crdroid-plus-kernel-build.yml   |  12 ++
 build-scripts/fix_kernelsu_49_v39.py          | 160 ++++++++++++++++++
 2 files changed, 172 insertions(+)
 create mode 100644 build-scripts/fix_kernelsu_49_v39.py

diff --git a/.github/workflows/crdroid-plus-kernel-build.yml b/.github/workflows/crdroid-plus-kernel-build.yml
index 45e2c09..fe6018e 100644
--- a/.github/workflows/crdroid-plus-kernel-build.yml
+++ b/.github/workflows/crdroid-plus-kernel-build.yml
@@ -437,6 +437,18 @@ jobs:
             fi
           fi
 
+          # CRITICAL FIX v39: FINAL umount fix - runs AFTER pre-patched file
+          # The pre-patched file still contains umount() calls that don't exist in kernel 4.9
+          echo "=== CRITICAL FIX v39: FINAL aggressive umount -> sys_umount fix ==="
+
+          if [ -f "$GITHUB_WORKSPACE/build-scripts/fix_kernelsu_49_v39.py" ]; then
+            echo "Running kernel 4.9 compatibility fix script v39 (FINAL)..."
+            KERNEL_DIR=~/kernel-build/android_kernel_oneplus_sdm845 python3 "$GITHUB_WORKSPACE/build-scripts/fix_kernelsu_49_v39.py" ~/kernel-build/android_kernel_oneplus_sdm845
+          elif [ -f "$GITHUB_WORKSPACE/fix_kernelsu_49_v39.py" ]; then
+            echo "Running kernel 4.9 compatibility fix script v39 from repo root..."
+            KERNEL_DIR=~/kernel-build/android_kernel_oneplus_sdm845 python3 "$GITHUB_WORKSPACE/fix_kernelsu_49_v39.py" ~/kernel_build/android_kernel_oneplus_sdm845
+          fi
+
           # Verification: Show line 591 of core_hook.c to verify the fix
           echo "=== Verifying core_hook.c line 591 (flags fix) ==="
           CORE_HOOK_FILE=$(readlink -f drivers/kernelsu/core_hook.c 2>/dev/null || echo "")
diff --git a/build-scripts/fix_kernelsu_49_v39.py b/build-scripts/fix_kernelsu_49_v39.py
new file mode 100644
index 0000000..2a11cf0
--- /dev/null
+++ b/build-scripts/fix_kernelsu_49_v39.py
@@ -0,0 +1,160 @@
+#!/usr/bin/env python3
+"""
+KernelSU-Next kernel 4.9 compatibility fix script v39
+FINAL FIX: Aggressive umount() -> sys_umount() replacement for kernel 4.9
+
+Issue: The pre-patched core_hook.c.patched file contains umount() calls
+which don't exist in kernel 4.9. This fix runs AFTER the pre-patched file
+is applied to ensure all umount calls are replaced.
+
+This fix is designed to be the FINAL and MOST AGGRESSIVE fix.
+"""
+
+import sys
+import os
+import re
+
+def aggressive_umount_fix(file_path):
+    """Aggressively replace all umount() calls with sys_umount()"""
+
+    if not os.path.exists(file_path):
+        return False, "File not found"
+
+    with open(file_path, 'r') as f:
+        content = f.read()
+
+    original_content = content
+    changes = []
+
+    # Pattern 1: Direct umount(something) -> sys_umount(something, 0)
+    # This is the most common pattern causing the undefined reference
+    pattern1 = r'(?<!sys_)(?<!try_)\bumount\s*\(\s*([^,)]+)\s*\)'
+    matches1 = list(re.finditer(pattern1, content))
+    if matches1:
+        content = re.sub(pattern1, r'sys_umount(\1, 0)', content)
+        changes.append(f"Replaced {len(matches1)} umount(X) calls with sys_umount(X, 0)")
+
+    # Pattern 2: Handle any remaining umount with two args (shouldn't exist but be safe)
+    pattern2 = r'(?<!sys_)(?<!try_)\bumount\s*\('
+    matches2 = list(re.finditer(pattern2, content))
+    if matches2:
+        changes.append(f"WARNING: Found {len(matches2)} remaining umount calls that need manual fix")
+        for i, match in enumerate(matches2):
+            # Get surrounding context
+            start = max(0, match.start() - 20)
+            end = min(len(content), match.end() + 40)
+            context = content[start:end]
+            changes.append(f"  Match {i+1}: ...{context}...")
+
+    # Check if any changes were made
+    if content != original_content:
+        with open(file_path, 'w') as f:
+            f.write(content)
+        return True, changes
+    else:
+        return False, "No umount() calls found to fix"
+
+
+def ensure_sys_umount_declaration(file_path):
+    """Ensure sys_umount is declared for kernel 4.9"""
+
+    with open(file_path, 'r') as f:
+        content = f.read()
+
+    # Check if already declared
+    if 'asmlinkage long sys_umount' in content or 'extern long sys_umount' in content:
+        return False, "sys_umount already declared"
+
+    # Add declaration after includes
+    declaration = '''/* Kernel 4.9 compatibility */
+asmlinkage long sys_umount(const char __user *name, int flags);
+'''
+
+    # Find last #include line
+    lines = content.split('\n')
+    insert_idx = 0
+    for i, line in enumerate(lines):
+        if line.startswith('#include'):
+            insert_idx = i + 1
+
+    lines.insert(insert_idx, declaration)
+    content = '\n'.join(lines)
+
+    with open(file_path, 'w') as f:
+        f.write(content)
+
+    return True, "Added sys_umount declaration"
+
+
+def process_core_hook_files(kernel_dir):
+    """Process all possible core_hook.c locations"""
+
+    possible_paths = [
+        os.path.join(kernel_dir, 'drivers/kernelsu/core_hook.c'),
+        os.path.join(kernel_dir, 'drivers/kernelsu/KernelSU/kernel/core_hook.c'),
+        os.path.join(kernel_dir, 'KernelSU/kernel/core_hook.c'),
+        os.path.join(kernel_dir, 'KernelSU-Next/kernel/core_hook.c'),
+    ]
+
+    results = []
+    for path in possible_paths:
+        real_path = os.path.realpath(path) if os.path.exists(path) else path
+        if os.path.exists(real_path):
+            print(f"\nProcessing: {real_path}")
+
+            # Apply aggressive umount fix
+            fixed, msg = aggressive_umount_fix(real_path)
+            if fixed:
+                print(f"  [FIXED] {msg}")
+            else:
+                print(f"  [INFO] {msg}")
+
+            # Ensure declaration exists
+            declared, msg = ensure_sys_umount_declaration(real_path)
+            if declared:
+                print(f"  [DECLARED] {msg}")
+
+            results.append((real_path, fixed, declared))
+
+    return results
+
+
+def main():
+    kernel_dir = os.environ.get('KERNEL_DIR', '')
+
+    if len(sys.argv) > 1:
+        kernel_dir = sys.argv[1]
+
+    if not kernel_dir:
+        print("Usage: KERNEL_DIR=/path/to/kernel python3 fix_kernelsu_49_v39.py [kernel_dir]")
+        sys.exit(1)
+
+    print("=" * 60)
+    print("KernelSU-Next 4.9 Compatibility Fix v39 (FINAL)")
+    print("=" * 60)
+    print(f"Kernel directory: {kernel_dir}")
+    print("")
+    print("This fix aggressively replaces ALL umount() calls with sys_umount()")
+    print("It runs AFTER the pre-patched file is applied.")
+    print("")
+
+    results = process_core_hook_files(kernel_dir)
+
+    print("\n" + "=" * 60)
+    print("SUMMARY")
+    print("=" * 60)
+
+    if results:
+        for path, fixed, declared in results:
+            status = "FIXED" if fixed else "OK"
+            print(f"  [{status}] {path}")
+        print("\nv39 fix applied. Build should now succeed.")
+        return 0
+    else:
+        print("  WARNING: No core_hook.c files found!")
+        print(f"  Searched in: {kernel_dir}")
+        return 1
+
+
+if __name__ == "__main__":
+    sys.exit(main())

From dc3fac503c071607b56377e4b36f438add81e172 Mon Sep 17 00:00:00 2001
From: Oracle Public Cloud User
 <opc@auranode.subnet09041427.vcn09041427.oraclevcn.com>
Date: Sat, 28 Mar 2026 14:26:05 +0000
Subject: [PATCH 2/2] Address CodeRabbit review feedback for v39 fix

- Fixed path typo: kernel_build -> kernel-build (underscore to hyphen)
- Added deduplication for core_hook.c file processing (seen_paths set)
- Renamed unused 'declared' variable to '_declared' to signal intentional discard
---
 .../workflows/crdroid-plus-kernel-build.yml   |   2 +-
 .../fix_kernelsu_49_v39.cpython-311.pyc       | Bin 0 -> 8189 bytes
 build-scripts/fix_kernelsu_49_v39.py          |  50 +++++++++++-------
 3 files changed, 31 insertions(+), 21 deletions(-)
 create mode 100644 build-scripts/__pycache__/fix_kernelsu_49_v39.cpython-311.pyc

diff --git a/.github/workflows/crdroid-plus-kernel-build.yml b/.github/workflows/crdroid-plus-kernel-build.yml
index fe6018e..98cc582 100644
--- a/.github/workflows/crdroid-plus-kernel-build.yml
+++ b/.github/workflows/crdroid-plus-kernel-build.yml
@@ -446,7 +446,7 @@ jobs:
             KERNEL_DIR=~/kernel-build/android_kernel_oneplus_sdm845 python3 "$GITHUB_WORKSPACE/build-scripts/fix_kernelsu_49_v39.py" ~/kernel-build/android_kernel_oneplus_sdm845
           elif [ -f "$GITHUB_WORKSPACE/fix_kernelsu_49_v39.py" ]; then
             echo "Running kernel 4.9 compatibility fix script v39 from repo root..."
-            KERNEL_DIR=~/kernel-build/android_kernel_oneplus_sdm845 python3 "$GITHUB_WORKSPACE/fix_kernelsu_49_v39.py" ~/kernel_build/android_kernel_oneplus_sdm845
+            KERNEL_DIR=~/kernel-build/android_kernel_oneplus_sdm845 python3 "$GITHUB_WORKSPACE/fix_kernelsu_49_v39.py" ~/kernel-build/android_kernel_oneplus_sdm845
           fi
 
           # Verification: Show line 591 of core_hook.c to verify the fix
diff --git a/build-scripts/__pycache__/fix_kernelsu_49_v39.cpython-311.pyc b/build-scripts/__pycache__/fix_kernelsu_49_v39.cpython-311.pyc
new file mode 100644
index 0000000000000000000000000000000000000000..71900f4fce439ae9eb5495e041cbc524714c1127
GIT binary patch
literal 8189
zcmcgxYit`=cAg=JoZ)atN)#navSg1QmPt#NV##*ade~NENs%A2*Rm5oL(rVDG&V(Q
zXNHzVkPBwv?b3n@DS;N@cAJ(dwplOI0>S=j(O-)IO@Gb=1C$t4fPt=l<R9C#Xpmn$
zclaDiPHHUB;pLh8IQO3Wm^t4$XMXE)(HLC+^T#iR4}BQ+@2F6|h*IX81Y|zKIP4C_
z;hZIc-@%Q$<&MR;Tklvo>oe>Q!4W`hoDHa*vjZi0iX$Ie&^Thd1>^BYj?$PIl*mt^
z@L9P6`xJ)~{EHWkah3(+AD}|NfNJ^@FGl&u!p-CJ{8O2IY?RoOgJ;-qY<VRl3lD^d
zAg{3xg{Q0(7KIg=eKK-}o|&7Uyv)wb-5O;l7Z*idl7uHb8()saqq0B19zV-UYf`XM
z7WtJ(D9kVOQJH-h6KllK^qeHc`BC=963?!P{Be*OUg9|rFY>{qSnTm&c(7deP>Ar*
z5S2qhRH{k~hawS)UR@HxODq?Q9+g@CsUXR$5Umk32oeOzSce#g-jg#ormwSdom3=&
z2DZ@3N<@HJ$T60WN^y~AK}Ja;Ysh0mP+3|RH%QZ!(V;iTOTuEb*!=--2q|t@D9W){
zt}fhQCue7`PcJOYy)%twG)RAqj%PkxIVUJl@VCPMn|}lP5hg>nd_Ai0<&3AGTwlH?
z>rP#QCu#wc$#Oe}CE*$P6#9Nv>3<4K;YmDKembkHkTX9~DGLWrCTBJG$FZoh2RngD
z#H!_~^)|MO<JfHshhNf?vMypNc#cyx*;yS}c2)C_DsxNPl65mPKfx(`(k_mh`l6jO
zt|YuA&>QyaGqiKYm9iv>T;=^J*P=UTmz%5Qq}?p%*BD2BY&F-x+e4+u|HB%{A6NsG
zq{M%E<-8oGc~Ry$z`W?a^O6tLtfa2T1?>LM@4-2z94T6EtMYNqB%P}{gKCTUyg4QT
z+RS{~DW}|7<u}dOVxBL^@KiagxjGpxqz&t2FiA_&nPh%$sZKiOdgZRUlMc><Mu8n_
zNV<SFerz+(1;;l2492uRHl7Y~vhI!Wn&p??h=%H%t2sDdcn3{-^|js!t|>`8M4H&Q
zfCZWpXR1mxvR2kAH7W;GYEkK|3i6UkwFWFR3HJ=ri_sX;t??+Q<Klc`$bW9U3+dnh
z%E;ncFmUfdQ6Jxvj@|PE@!x%KAaK7wfXabHV~xVTcVd8jBam=hFRE>hP4tzu?X6eM
z<d)Ip@DGShc;217K0h}<JIc<Wxw44?Sne|Db0NBD*01tXNM@rvXwl_RG#-i|MJXb6
z^)(<W_O%~Kf@0Y#2rt;Lkw#3|M%lr^!Gslvj<4Q8dvj<$R-Sr!=Ef!=%?<+w8g)xd
z(g{GACEaeo29#ned{ieyKE&xn1dv3xh`dfc6r!9U^P+B*;tzB?Km-q)ZH)l0gOA3S
zc`+pOx^+49RJTcTNR%NjM0G3TvaO2HHQ?4A2w{STIQ*ebA}|9`)Ei==uqZ@Bkzl!o
z3@?SEi@c;$zzkBOCv<Arz(xsjrMlrt-LQ@ey3Jq^TVOB5Rv@MEpURROO04PnZfr<i
zUK!%Vn7~OZ0M63!2XP_74H@)!5$mNP*rQ<4D3Id8lV^ee{(~!PdUL3P`X!VPqT@LV
zVjAHut-wL0vH!MXjC<qF&9m!gvjNrFt2uiM7;Zb7_w*^AzFmhaBV?OYN2liKMD+)A
zjd^d=M|~gmec<2rXUKvT>Wl`1{0CzBFGQ+!`{lm<_S^lc3wEsautNCq?uPVifx;Tz
z!qac>c$$9s*7l_w_2Q`N8`gZos%J#=jHKVrlb!-*vE9LUyuR(eY^&1q=06-zy_1@E
zGDGGm&!&6btsJ=Zw<EcypG^E_0<y)VQnxkgwnE(oaSg*iB!IM?t&GL}ebb9uzw7+F
zPUQesOe)1|6t7VHPHR`rueSDUt^Ebe<~VDR=Wk~wGx1&Dfh?Xq_19y6G4`v8znpk6
ztR6hB`UW-MU}k#9+x*LL_UH$Z?MQC=w{w3p_sON-T>89Q?LDb_Pifv$nW=nBXO_=R
zWd*IppP9`wO`G%U^UA^TT<8V<Nf<JaJ|~NJpei$=F%ycBV11NF@pgYjb}MA}j;nDi
zkX=w+U7D*4cHhyN`*EIeZO*RGZq2GphsJa$Oa~mOandCM2kPjA1Ffwy`qq)+f$rgY
z>1o<}H<MDRj+dU(is!V{15e)Xn<tyBe`q2m50HOo$ASLySj%ju^~+9Twu}6-7YAyx
zvh2_Yp}{x*2^s{ftcqo2*hu)kh)}}=JxiAv&%j19e&+tKwvjk%(((jsCyqdO(ZbmP
z8|*2l-QU9FNHhL`bwz=7Rk5OA_Jo-OtboIY6<|-1NpgQXk8_y&h_<AiqY<X2Y#S^f
zudTe=+B;Zmdu&0D%%Pp#bJfo(WvSb7r+LRp7+kZi>X7?j@aye41K8_I3``dd{3`yj
zh6x{zfQ8T1xho>FkLMHZA!#`xL?4G1c{UOQ<f?W~w16K4>bkxjfziP#8-P2)mo@Me
zih>fsp<`^(8QaJGIyOYVTBIK^1Kb?|43P~6;}S2j$D*NSegHfmun8ieMJX^we}~fs
zd?u(KAsUXvIbKBDDkAGSV7Za#nB+KEZ_N~oQv)9`77b7$gP4)I2<QgRkt`w@)@cLj
zBxIN9cJRu0fIZ?-Oo)oeXoGI;sM|qAIuO6JW>DP*dk6?e7yAVT?x~30VFbERk0w@z
zEU^#OcNTrIV0Dhcn&HG@;2DL#v<`2mfj+eR{}o{<jOxv;D&!za!x#T$Q5n7oDeoCk
zJR=3WrHM#iD$tnAw|Qm#O14*J4r|Qe^mM+_o4&Zy)RDcZHuY#tJqUOX8{|2e!80fG
zPS57K^>f)#)!C;x`;c}#oNsH-{`gnTnTy}PBpkoE{Iknj?b*935zvT$LIm<|@20RW
zWNoVZkmf#=p524BZGg3uW0PDbw@8(0)u>j5YDIwC2E&&K0^BwPxHWav)@rT<xO=z&
z{ohcWL((BQlMkAQt-mKulmCdHCcmItrUtBE3=mUqkY9}8KuszcJ(Y$^E`Sm7x{fbb
zr?OwqZLW8UU$b|e8hp*(gjw}15-A+qW=pM7uLBut5(l;Z#}x4jN&7nU^(t7xH@aWZ
ze}8pf2P`QY0ssIld}AhQeTCZJ<A}*MP9Zq$wqIi(?{TSQWJ;E<WZeiAD6@-Arm^fv
za=)>nUK1BU;C@_kQ&}DW=zbEx{Rh|GJQlV8SODEd5U@I8F$cS4Pf`!}gcde-li&~r
z9Epi*Vhqf22tq-~X(a}swFgGns}c_bXbBrySz--;5Y%=YG;*;n(ZPujixZ`xve`aV
z6G}@A>}f4x&ce+Szlr<c_cfN#suVihue*ybEd`vrQ8q!d>|F>MPG7jsCLEBTo1eJ~
zSr=q4Oix{&ypCD|P7y_GM6`F}aUkGs@v?~GMmhzM88Lh(Yls7<YDENqGn@`X_lsU&
z3OIDSxbh$;h`PJHfrif|=_JG|;}ID^N8<TtvC@#GJJ8@o&&&qC3=gi|vb3mEWq_8@
zw90tI76IwNO7W<`Yb%`&xIqzK63+wgZTL&+A{0@W!?{6jGV9FN9i{7>N}boJ^E(t>
zt-GpH*EH&ysqU;woztjurn)JWx}Z@P3JzO~Jw0D=W1hzJ>`tTagXZn#^u?DAhYDDm
z?UX?>#Ez$NV|DYV>p#tMs;5Wu0JPwa<~-xu^|frz{;RL|D_?K!hUz<}`Hp2AVA3<q
zSz2*+LjtBe<wb7J)|yInX;ha&b?s6v#eFQh1_Y8y^=nkWLiK~6*V6yujM{ukYd-ay
z&e$?i-q}#Vtd1i)OrzrM%}PKZsmu|LIieV87q#?c!$2UZOs~fDDoiho)pq)GL2W&+
zwVr?O%1~Q9d1n)vcfc6yXb#wqLQ)yO#`qP+|FY<fWgWRwFZ{}Z5tTZrQ7086y>xoh
z^WVThU|~ecR|R4i0Gii>7!;c~E9#UfvMN`ikr?)<qCm}dHD`TnsAf<k`*L&Enur&~
zYNejqcqdn<fN@N&iEmaTsSu%shp#q9Wg4YUy}>9ou|Tda8d}%-{e8(K#NhT(=e6VS
zRra;5brH-u>2+R8UF+VFYhk1=GM>UstERtq$DH5Wx@S+4xOmC{=&pP4l%*gFo+6S~
z@a>o=&bh!_t6i}hK_KTzT43k)oG$=$EpJ2pJh(=P?$>g7>p4uVP5Yk@X9f82&Fl1~
z>Fe{;mxCAPuIrfLBP6cel)%*)WveX{Lr4z~$+42&XIIwbrC4+X1=aQm7_oP&I{AKL
zeBw{I5<_KBDt@v;->Rle->PQdTNT8qSikYnC6MqIJFtKUd{~Z&Yk);lmG3pxNLDrU
z3ZI8AU#^cE?uor72Ipk<2R<etU$XQuX)rNC{|W1!(8-0HSFTK6zpdL06xA(PFD0x1
zo(T(}Xg`gX47*<u$G(N)d91V)i^DA%TV<tqI1B;R!J;47$Fh}REJR5w5H=WwVF1L6
z(d4d#hh-P|kZ63B6QZMRz+oV?PV&(wf*6Ac=OO?!vYY^_*C2cv5*MF{Z=x~*5mng$
z!+>4<5!8stKPzJU1Y&TM95_H*J9v_ET#``Spa`=?j3rQXSiB0g&){G3?RFZQw%b2C
z^x>f#^WuTpctUGDk-oS~(tkd(@uT-s&r(}I`HE~+$kv_a*6mx_(DvO7{nFWtd@A1#
z!m#@)^`1t(r%><hP|gi%tMR#W!<ljJ5{``j{j1NermyZ04uu|5iE)h>SBP;anp*m^
z35D*5q!I%fF`y6wh@<=cxo30fIiv4YmAIx6*A(KKQ5;o?F^w2gh_M>RUH6gPBh`IE
zbDsbo4F(ueiFS=>SBQ4R6;KJEM))9TiAtr3y(HZb|Fbpb`;NnU9LEfzGToUATSu~O
zdG?6XJhb)h*1IoGW@Ze;^k?mg*AFQN5RC3B6VRA|VkBcX##L%Uqb3w;BJXL)`~)VV
z(8DS*q7fqsF=9;LpS`Hien?dx)5&B`h?D3PsbCNtQZN{xbut*_V&Py=ya+8tQx2K>
zHCEbrR7B^A`k^=lFkgn^?=WLD<J7;S#+PGUJi?z9Z$J~^7$*G{5b#iOyg)c`qJSaN
zg5l0IjedE|kv4vnOo8ab-z;Diavkr$-zZ=ea@LQ7T~Q(L;N7^ZfK|vv93z-a;wx*L
lVr|P4OnU0s+nF1g8_(a_c&BnMa5Rt#Rt(O}+GfoEe*qH#b9n#&

literal 0
HcmV?d00001

diff --git a/build-scripts/fix_kernelsu_49_v39.py b/build-scripts/fix_kernelsu_49_v39.py
index 2a11cf0..e79071f 100644
--- a/build-scripts/fix_kernelsu_49_v39.py
+++ b/build-scripts/fix_kernelsu_49_v39.py
@@ -87,7 +87,7 @@ def ensure_sys_umount_declaration(file_path):
 
 
 def process_core_hook_files(kernel_dir):
-    """Process all possible core_hook.c locations"""
+    """Process all possible core_hook.c locations with deduplication"""
 
     possible_paths = [
         os.path.join(kernel_dir, 'drivers/kernelsu/core_hook.c'),
@@ -97,24 +97,34 @@ def process_core_hook_files(kernel_dir):
     ]
 
     results = []
+    seen_paths = set()  # Track processed real paths to avoid duplicates
+
     for path in possible_paths:
-        real_path = os.path.realpath(path) if os.path.exists(path) else path
-        if os.path.exists(real_path):
-            print(f"\nProcessing: {real_path}")
+        if not os.path.exists(path):
+            continue
+
+        real_path = os.path.realpath(path)
+
+        # Skip if we've already processed this real file
+        if real_path in seen_paths:
+            continue
+        seen_paths.add(real_path)
 
-            # Apply aggressive umount fix
-            fixed, msg = aggressive_umount_fix(real_path)
-            if fixed:
-                print(f"  [FIXED] {msg}")
-            else:
-                print(f"  [INFO] {msg}")
+        print(f"\nProcessing: {real_path}")
 
-            # Ensure declaration exists
-            declared, msg = ensure_sys_umount_declaration(real_path)
-            if declared:
-                print(f"  [DECLARED] {msg}")
+        # Apply aggressive umount fix
+        fixed, msg = aggressive_umount_fix(real_path)
+        if fixed:
+            print(f"  [FIXED] {msg}")
+        else:
+            print(f"  [INFO] {msg}")
 
-            results.append((real_path, fixed, declared))
+        # Ensure declaration exists
+        declared, msg = ensure_sys_umount_declaration(real_path)
+        if declared:
+            print(f"  [DECLARED] {msg}")
+
+        results.append((real_path, fixed, declared))
 
     return results
 
@@ -145,15 +155,15 @@ def main():
     print("=" * 60)
 
     if results:
-        for path, fixed, declared in results:
+        for path, fixed, _declared in results:
             status = "FIXED" if fixed else "OK"
             print(f"  [{status}] {path}")
         print("\nv39 fix applied. Build should now succeed.")
         return 0
-    else:
-        print("  WARNING: No core_hook.c files found!")
-        print(f"  Searched in: {kernel_dir}")
-        return 1
+
+    print("  WARNING: No core_hook.c files found!")
+    print(f"  Searched in: {kernel_dir}")
+    return 1
 
 
 if __name__ == "__main__":