Another Update for my Minidlna/Samba3/USB/FTP DD-WRT Mod

4 posts / 0 new
Last post
Kong
Kong's picture
Another Update for my Minidlna/Samba3/USB/FTP DD-WRT Mod

Hi guys,

 

after a few days testing, there is a new Build of my dd-wrt mod.

New:

  • added webif setting for minidlna to allow db update interval setting, default is 60s
  • compile optimized
  • improve wan-lan routing performance

Fixed:

  • large filesupport now working (tested) for samba and busybox console utils(ls,rm etc.)

Removed:

  • nfs modules, due to image size, eventually I'll add them in a seperate build + nfs-utils

Known Issue:

  • proftpd reports wrong file size for files >4GB, this does not affect the actual transfer of files >4GB

Download

 

@peter, as always please update the file in the download section

Kevin Mahoney
Kevin Mahoney's picture
Hey Kong,

Hey Kong,

Looks like there is an error in your startup script for samba3.

if [ "$(nvram get samba3_usr2)" != "" ]; then
grep -q $USR2 /etc/passwd || echo $USR1":*:1000:1000:"$USR2":/var:/bin/false" >> /etc/passwd
smbpasswd $USR2 $PASS2
fi

Should be?

if [ "$(nvram get samba3_usr2)" != "" ]; then
grep -q $USR2 /etc/passwd || echo $USR2":*:1001:1001:"$USR2":/var:/bin/false" >> /etc/passwd
smbpasswd $USR2 $PASS2
fi

Kong
Kong's picture
Good catch!

Good catch!

I'll update it. Actually I think I should change those edit fields and use a text box with user:pass style as in proftpd. That would allow to add more users.

Kevin Mahoney
Kevin Mahoney's picture
Some feedback:

Some feedback:

- I like the idea about adding additional users however I don't like how the password is displayed in plain text. We really need a whole interface maybe as a part of sshd configuration for handling extra users. Having root exposed to the world isn't the best security measure.
- Make a simlink from /etc/samba/smb.conf -> /tmp/etc/smb.conf and use /etc/samba/smb.conf
- Grepping /etc/passwd for a user name is proned to problems
if user10 existed in /etc/passwd
grep user1 would return 0 and user10 wouldn't be added to /etc/passwd