forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit6d306ab
committed
Fix get_dirent_type() for Windows junction points.
Commit87e6ed7 added code that intended to report Windows "junctionpoints" as DT_LNK (the same way we report symlinks on Unix). Windowsjunction points are *also* directories according to the Windowsattributes API, and we were reporting them as as DT_DIR. Change theorder we check the attribute flags, to prioritize DT_LNK.If at some point we start using Windows' recently added real symlinksand need to distinguish them from junction points, we may need torethink this, but for now this continues the tradition of wrapperfunctions that treat junction points as symlinks.Back-patch to 14, where get_dirent_type() landed.Reviewed-by: Michael Paquier <michael@paquier.xyz>Reviewed-by: Alvaro Herrera <alvherre@alvh.no-ip.org>Discussion:https://postgr.es/m/CA%2BhUKGLzLK4PUPx0_AwXEWXOYAejU%3D7XpxnYE55Y%2Be7hB2N3FA%40mail.gmail.comDiscussion:https://postgr.es/m/20220721111751.x7hod2xgrd76xr5c%40alvherre.pgsql1 parent63a8c68 commit6d306ab
1 file changed
+10
-6
lines changedLines changed: 10 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
106 | 106 |
| |
107 | 107 |
| |
108 | 108 |
| |
109 |
| - | |
110 |
| - | |
111 |
| - | |
112 |
| - | |
113 |
| - | |
114 |
| - | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
115 | 117 |
| |
| 118 | + | |
| 119 | + | |
116 | 120 |
| |
117 | 121 |
| |
118 | 122 |
| |
|
0 commit comments
Comments
(0)