FreeBSD 13 כמעט הסתיים עם הטמעה חריפה של WireGuard עם הפרות רישיון ופגיעויות

מבסיס הקוד שעליו נוצרה מהדורת FreeBSD 13, הקוד המיישם את פרוטוקול ה-VPN של WireGuard, שפותח בפקודת Netgate ללא התייעצות עם מפתחי ה-WireGuard המקורי, וכבר נכלל במהדורות היציבות של הפצת pfSense, היה שערורייתי. הוסר. לאחר סקירת קוד על ידי ג'ייסון א. דוננפלד, מחבר ה-WireGuard המקורי, התברר שהיישום המוצע של FreeBSD של WireGuard היה חתיכת קוד עלוב, גדוש בהצפת חיץ ומפר את ה-GPL.

היישום הכיל פגמים קטסטרופליים בקוד ההצפנה, חלק מפרוטוקול ה-WireGuard הושמט, היו שגיאות שהובילו לקריסת הליבה ולעקוף שיטות אבטחה, וכן נעשה שימוש במאגרים בגודל קבוע לנתוני קלט. נוכחותם של סתימות במקום צ'קים שתמיד מחזירות "true", כמו גם פרינט באגים שנשכחו ב-printfs עם פלט של פרמטרים המשמשים להצפנה, והשימוש בפונקציית השינה למניעת תנאי מרוץ אומרים הרבה על איכות הקוד.

חלקים מסוימים של הקוד, כמו פונקציית crypto_xor, הועברו מהטמעת WireGuard שפותחה עבור לינוקס, תוך הפרה של רישיון GPL. כתוצאה מכך, ג'ייסון דוננפילד, יחד עם קייל אוונס ומאט דאנוודי (מחבר ה-WireGuard port עבור OpenBSD), לקחו על עצמם את המשימה של עיבוד מחדש של היישום הבעייתי ותוך שבוע, החליפו לחלוטין את כל הקוד של המפתח שנשכר על ידי Netgate . הגרסה ששונתה שוחררה כסט נפרד של תיקונים, הוצבה במאגר הפרויקטים של WireGuard ועדיין לא נכללה ב-FreeBSD.

מעניין שבהתחלה לא היו סימנים לבעיות; Netgate, שרצתה להיות מסוגלת להשתמש ב-WireGuard בהפצת pfSense, שכרה את מתיו מייסי, הבקיא בקרנל FreeBSD ובמחסנית הרשת, מעורב בתיקוני באגים ובעל ניסיון בפיתוח מנהלי התקן רשת עבור מערכת הפעלה זו. מייסי קיבלה לוח זמנים גמיש ללא מועדים או בדיקות אמצע. מפתחים שפגשו את מייסי בזמן העבודה על FreeBSD תיארו אותו כמתכנת מוכשר ומקצועי שלא עשה יותר טעויות מאחרים והגיב כראוי לביקורת. האיכות הירודה של קוד היישום WireGuard עבור FreeBSD הפתיעה אותם.

לאחר 9 חודשי עבודה, מייסי הוסיף את היישום שלו לסניף HEAD, ששימש לגיבוש המהדורה של FreeBSD 13, בדצמבר האחרון ללא השלמת סקירת עמיתים ובדיקות. הפיתוח נעשה ללא תקשורת עם מפתחי ה-WireGuard המקורי וה- יציאות OpenBSD ו-NetBSD. בפברואר, Netgate שילבה את WireGuard במהדורה היציבה של pfSense 2.5.0 והחלה לשלוח חומות אש המבוססות עליה. לאחר שזוהו בעיות, קוד ה-WireGuard הוסר מ-pfSense.

הקוד שנוסף חשף נקודות תורפה קריטיות שהיו בשימוש בניצול של 0 ימים, אך בתחילה Netgate לא הכירה בקיומן של נקודות תורפה וניסתה להאשים את המפתח של ה-WireGuard המקורי בהתקפות והטיות, מה שהשפיע לרעה על המוניטין שלה. מפתח הפורט דחה תחילה טענות לגבי איכות הקוד וראה אותן מוגזמות, אך לאחר שהדגים שגיאות, הוא הפנה את תשומת הלב לעובדה שהבעיה החשובה באמת היא היעדר סקירה נכונה של איכות הקוד ב-FreeBSD, מכיוון שהבעיות נותרו בלתי מזוהות במשך חודשים רבים. (נציגי Netgate ציינו שהסקירה פומבית הושקה עוד באוגוסט 2020, אך מפתחי FreeBSD בודדים ציינו שב-Phabricator הסקירה נסגרה על ידי Macy ללא השלמה ועם התעלמות מהערות). צוות הליבה של FreeBSD הגיב לתקרית בהבטחה לחדש את תהליכי סקירת הקוד שלהם.

מתיו מייסי, המפתח של יציאת FreeBSD הבעייתית, התייחס למצב באומרו שהוא עשה טעות גדולה כשלקח על עצמו את העבודה מבלי שהיה מוכן ליישם את הפרויקט. מייסי מסבירה את התוצאה בשחיקה רגשית ובתוצאה של בעיות שנוצרו עקב תסמונת פוסט-קוביד. יחד עם זאת, מייסי לא מצא את הנחישות לנטוש את ההתחייבויות שכבר לקח על עצמו וניסה להביא את הפרויקט לסיומו.

ייתכן שמצבו של מייסי הושפע גם מעונש מאסר שעבר לאחרונה על ניסיון פינוי בלתי חוקי של דיירים מבית שרכש שלא היו מוכנים לצאת מרצונם. במקום זאת, הוא ואשתו ניסרו את קורות הרצפה ושברו חורים ברצפות כדי להפוך את הבית לבלתי ראוי למגורים, וכן ניסו להפחיד את התושבים, פרצו לדירות מאוכלסות והוציאו את חפציהם (הפעולה סווגה כפריצה). כדי להימנע מאחריות למעשיו, מייסי ואשתו ברחו לאיטליה, אך הוסגרו לארצות הברית וריצו יותר מארבע שנות מאסר.

מקור: OpenNet.ru