First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 117546
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Mobile Herd <mobile@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Carlo Marcelo Arenas Belon <carenas@sajinet.com.pe>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
ndiswrapper-reinstall.patch "obvious" patch to the problem patch Carlo Marcelo Arenas Belon 2006-01-03 01:02 0000 532 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 117546 depends on: Show dependency tree
Show dependency graph
Bug 117546 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2006-01-02 23:45 0000
as shown by :

 * Attempting to automatically reinstall any Windows drivers
 * you might already have.
 * Driver: netbc564
mv: cannot overwrite directory `/tmp/netbc564'
netbc564 is already installed. Use -e to remove it

how to reproduce :

  `module-rebuild rebuild` twice

problems :

* mv won't succeed if another directory with the same name already exists
* if forced, it will lead to a security problem (race condition)
* assumes /tmp is available as a TMPDIR
* leaves /tmp dirty

------- Comment #1 From Carlo Marcelo Arenas Belon 2006-01-03 01:02:22 0000 -------
Created an attachment (id=76051) [edit]
"obvious" patch to the problem

still problematic as :

0) not tested
1) not using a real lock and therefore has a race condition between the test
and the mv/rm
2) probably still violates the sandbox rules by using ${TMPDIR}

------- Comment #2 From Petteri Räty 2006-06-22 05:33:57 0000 -------
(In reply to comment #1)
> Created an attachment (id=76051) [edit]
> "obvious" patch to the problem
> 
> still problematic as :
> 
> 0) not tested
> 1) not using a real lock and therefore has a race condition between the test
> and the mv/rm
> 2) probably still violates the sandbox rules by using ${TMPDIR}
> 

Well I think I fixed this by switching to using ${T} that should be used within
ebuilds. Please test ndiswrapper-1.17 and reopen if not fixed well enough. I
don't have the hardware at hand atm so I can't test.

------- Comment #3 From Carlo Marcelo Arenas Belon 2006-06-23 09:07:48 0000 -------
confirmed: works like a charm, thanks

First Last Prev Next    No search results available      Search page      Enter new bug